[clang] [C++20][Modules] Implement P1857R3 Modules Dependency Discovery (PR #107168)
via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 19 06:09:48 PST 2025
yronglin wrote:
> I see that CWG2947 test is unusual, because it take 9 RUN lines to test C++20 mode. I suggest you to group tests cases in fewer files, which should reduce the number of RUN lines. Then I'd like to see C++23 and C++26 modes tested.
> Then I'd like to see C++23 and C++26 modes tested.
Done.
> I suggest you to group tests cases in fewer files, which should reduce the number of RUN lines.
Hmm, I kept 9 RUN lines and repeat it for C++23/26, because module/import directive lexical errors may cut-off lexing stage, so multiple module directive in the same file may impact the diagnostics, What do you think?
https://github.com/llvm/llvm-project/pull/107168
More information about the cfe-commits
mailing list