[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