[all-commits] [llvm/llvm-project] 849f1d: [XRay] Rename XRayOmitFunctionIndex to XRayFunctio...

Fangrui Song via All-commits all-commits at lists.llvm.org
Sun Jun 11 15:27:35 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 849f1dd15e92fda2b83dbb6144e6b28b2cb946e0
      https://github.com/llvm/llvm-project/commit/849f1dd15e92fda2b83dbb6144e6b28b2cb946e0
  Author: Fangrui Song <i at maskray.me>
  Date:   2023-06-11 (Sun, 11 Jun 2023)

  Changed paths:
    M clang/include/clang/Basic/CodeGenOptions.def
    M clang/include/clang/Driver/Options.td
    M clang/lib/CodeGen/BackendUtil.cpp
    M clang/lib/Driver/XRayArgs.cpp
    A clang/test/CodeGen/xray-function-index.c
    R clang/test/Driver/XRay/xray-function-index-flags.cpp
    A clang/test/Driver/xray-function-index.cpp
    M llvm/include/llvm/CodeGen/CommandFlags.h
    M llvm/include/llvm/Target/TargetOptions.h
    M llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
    M llvm/lib/CodeGen/CommandFlags.cpp
    M llvm/test/CodeGen/AArch64/xray-omit-function-index.ll

  Log Message:
  -----------
  [XRay] Rename XRayOmitFunctionIndex to XRayFunctionIndex

Apply my post-commit comment on D81995. The negative name misguided commit
d8a8e5d6240a1db809cd95106910358e69bbf299 (`[clang][cli] Remove marshalling from
Opt{In,Out}FFlag`) to:

* accidentally flip the option to not emit the xray_fn_idx section.
* change -fno-xray-function-index (instead of -fxray-function-index) to emit xray_fn_idx

This patch renames XRayOmitFunctionIndex and makes -fxray-function-index emit
xray_fn_idx, but the default remains -fno-xray-function-index .




More information about the All-commits mailing list