qiongsiwu wrote: > LGTM, thanks! FWIW I think this is easy to test even upstream by scanning two > TUs with different defines that canonicalize to the same thing, and then > checking the number of entries in the module cache. Might be worth doing here.
Ah great suggestion! Thanks! A test is now added here as well. https://github.com/llvm/llvm-project/pull/159620 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
