https://github.com/jansvoboda11 approved this pull request.
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. https://github.com/llvm/llvm-project/pull/159620 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
