[all-commits] [llvm/llvm-project] c7dce0: [Driver] Adjust -fsanitize=function & -mexecute-on...
Fangrui Song via All-commits
all-commits at lists.llvm.org
Wed Aug 30 20:30:26 PDT 2023
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: c7dce0283b5b502dacd47eb1488342a31f4ed35a
https://github.com/llvm/llvm-project/commit/c7dce0283b5b502dacd47eb1488342a31f4ed35a
Author: Fangrui Song <i at maskray.me>
Date: 2023-08-30 (Wed, 30 Aug 2023)
Changed paths:
M clang/include/clang/Basic/Sanitizers.h
M clang/lib/Basic/CMakeLists.txt
M clang/lib/Basic/Sanitizers.cpp
M clang/lib/Driver/SanitizerArgs.cpp
M llvm/include/llvm/Option/ArgList.h
M llvm/lib/Option/ArgList.cpp
M utils/bazel/llvm-project-overlay/clang/BUILD.bazel
Log Message:
-----------
[Driver] Adjust -fsanitize=function & -mexecute-only interop after D158614
clangDriver depends on clangBasic, so clangBasic should not depend on
clangDriver, even just its header. Also remove clangBasic's dependency
on LLVMOption.
The issue can be seen through the bazel commit
d26dd681f9726ed7d43d7c0bdd8ee3cb2db69a2b which is reverted now.
Add hasFlagNoClaim and use it as we don't want to suppress
-Wunused-command-line-argument for -mexecute-only just because
-fsanitize= is specified.
More information about the All-commits
mailing list