[clang] [clang][CodeGen] Always use CLANG_VENDOR as a quoted string (PR #75935)

Shoaib Meenai via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 19 11:40:18 PST 2023


https://github.com/smeenai requested changes to this pull request.

IIRC, applying definitions like this only to a small set of files was an intentional decision, to minimize rebuilding when their values changed. It matters much more for the commit hash than the vendor, but passing defines like this to every single compilation in Clang is ugly IMO. Can we instead add a function to Version.cpp to access the vendor (so that the correct type gets encoded in the function signature) and use that from CodeGenModule.cpp instead of referencing `CLANG_VENDOR` directly?

https://github.com/llvm/llvm-project/pull/75935


More information about the cfe-commits mailing list