On Thu, Feb 16, 2023 at 08:51:00PM -0700, Rebecca Cran wrote: > In https://bugzilla.tianocore.org/show_bug.cgi?id=2842 clang support was > added by having users specify "make CXX=llvm" when building BaseTools. > > Improve the detection of when a user wants to use the clang toolchain: > instead of checking if CXX=llvm (which in most cases doesn't make sense, > because the C++ compiler won't be run via an 'llvm' command), run > '$(CC) --version | grep clang' to see if the compiler's version string > contains 'clang', and if so configure the environment. > > This provides flexibility to specify for example CC=clang-17 > CXX=clang++-17 if multiple versions are installed.
That totally makes sense. And maybe simply use 'cc' by default then? take care, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#101226): https://edk2.groups.io/g/devel/message/101226 Mute This Topic: https://groups.io/mt/97022153/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
