This is a patch series that I'd really like to get committed.
It'll allow us to drop the dependency on gcc for building edk2 on FreeBSD.
The Github branch is https://github.com/bcran/edk2/tree/mdepkg-c11 and
there's a PR at https://github.com/tianocore/edk2/pull/4142.
--
Rebecca Cran
On 3/9/23 8:47 AM, Rebecca Cran wrote:
Could I get some reviews on this please?
Thanks.
Rebecca Cran
On 2/16/23 8:50 PM, Rebecca Cran wrote:
Currently, the BaseTools Makefiles use BUILD_CC and BUILD_CXX, which
doesn't allow users to override the compiler to use in the expected way
by running e.g. "make CC=clang-17 CXX=clang++-17". clang/llvm support
was added in https://bugzilla.tianocore.org/show_bug.cgi?id=2842 in a
way that required users to run "make CXX=llvm" and have clang and
clang++
executables under $(CLANG_BIN). As far as I know this isn't a standard
way of telling a build system to use clang, and so is likely difficult
to discover by users.
This patch series fixes that, and as a side effect allows the clang
analyzer to run via "scan-build make".
Since clang 17 defaults to C++17 or newer where the 'register' keyword
is deprecated and the warning turned into an error, override the
version used when building C++ code via "-std=c++14".
Rebecca Cran (3):
BaseTools: Allow users to specify compiler to use with make CC= CXX=
BaseTools: Improve detection of users wanting to build using clang
BaseTools: Build against C++14 when building with clang
BaseTools/Source/C/DevicePath/GNUmakefile | 7 ++-
BaseTools/Source/C/LzmaCompress/GNUmakefile | 2 +-
BaseTools/Source/C/Makefiles/app.makefile | 2 +-
BaseTools/Source/C/Makefiles/footer.makefile | 6 +-
BaseTools/Source/C/Makefiles/header.makefile | 59
++++++++++----------
BaseTools/Source/C/VfrCompile/GNUmakefile | 19 ++++---
BaseTools/Source/C/VfrCompile/Pccts/antlr/makefile | 20 +++----
BaseTools/Source/C/VfrCompile/Pccts/dlg/makefile | 31 +++++-----
BaseTools/Source/Python/Workspace/DscBuildData.py | 2 +-
9 files changed, 76 insertions(+), 72 deletions(-)
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#101165): https://edk2.groups.io/g/devel/message/101165
Mute This Topic: https://groups.io/mt/97022151/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
