-
Notifications
You must be signed in to change notification settings - Fork 3.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] [3/5] support C++20 Modules, add deps-scanner #22429
base: master
Are you sure you want to change the base?
[WIP] [3/5] support C++20 Modules, add deps-scanner #22429
Conversation
9582891
to
4f31b5a
Compare
4f31b5a
to
c6a1120
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After filtering out the other 2 PRs this change looks mostly reasonable. I posted some nits on the files unique to this change.
src/main/java/com/google/devtools/build/lib/rules/cpp/CcModule.java
Outdated
Show resolved
Hide resolved
c6a1120
to
546b0b7
Compare
I just keep the main commit in this patch. I also fix |
I split the XXL PR #19940 into several small patches.
This is the third patch of Support C++20 Modules, I add
deps-scanner
tool and update toolchainsUsing C++20 Modules means that compilation units must be compiled in topological order. see Discovering Dependencies.
Considering the various compilers, I add the tool
deps-scanner
. The default implementation is a script wrapper that uses different scanning methods depending on the compiler. Here is a demo on how to scan C++20 dependencies. https://github.com/PikachuHyA/cpp20_modules_scan_dependency_demoI also update the toolchain, apply different flags to different actions.