Skip to content

rebuild passes_list.md does not function in OSS #2556

@allight

Description

@allight

Describe the bug

Post #2531, The new passes_list.md file is meant to be automatically regenerated using the //xls/passes/tools:rebuild_documentation target. Unfortunately due to differences in how the build works between google and oss this does not work in the oss version. This is due to inside google the libcxx being a normal dependency resolved by bazel but externally it uses the compiler to provide this library. This causes issues since the libtooling binary that rebuild_documentation uses internally does not know how to find the library (and the libcxx is tightly bound to the exact clang version so it might not work even if it did find it).

If libcxx were buildable in bazel this would probably be fixable by manually adding it to the search paths like we do for some of the other libc headers. Unfortunately there is no OSS bazel build for libcxx.

It's likely this could be hacked together by building libcxx separately and pointing rebuild_benchmarks to its files.

For now however the rebuild command will act like the bazel rule doc where the generation is done internally at google until somebody has the time to fix this up.

Metadata

Metadata

Assignees

No one assigned

    Labels

    code-maintenanceEverything w.r.t. code quality, maintenance, TODOsdocumentationImprovements or additions to documentationestimate:MMedium: ~1-3 daysgood first issueGood for newcomers

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions