ianlevesque marked 2 inline comments as done.
ianlevesque added inline comments.
Comment at: clang/include/clang/Basic/CodeGenOptions.def:114
+///< Set with -fno-xray-function-index to omit the index section.
+CODEGENOPT(XRayOmitFunctionIndex , 1, 0)
+
MaskRay wr
MaskRay added inline comments.
Comment at: clang/include/clang/Basic/CodeGenOptions.def:114
+///< Set with -fno-xray-function-index to omit the index section.
+CODEGENOPT(XRayOmitFunctionIndex , 1, 0)
+
ianlevesque wrote:
> MaskRay wrote:
> > ianlevesque wrote:
>
ianlevesque marked 4 inline comments as done.
ianlevesque added inline comments.
Comment at: clang/include/clang/Basic/CodeGenOptions.def:114
+///< Set with -fno-xray-function-index to omit the index section.
+CODEGENOPT(XRayOmitFunctionIndex , 1, 0)
+
MaskRay wr
MaskRay added inline comments.
Comment at: clang/include/clang/Basic/CodeGenOptions.def:114
+///< Set with -fno-xray-function-index to omit the index section.
+CODEGENOPT(XRayOmitFunctionIndex , 1, 0)
+
ianlevesque wrote:
> MaskRay wrote:
> > Nit: a variable name
ianlevesque marked 3 inline comments as done.
ianlevesque added inline comments.
Comment at: clang/include/clang/Basic/CodeGenOptions.def:114
+///< Set with -fno-xray-function-index to omit the index section.
+CODEGENOPT(XRayOmitFunctionIndex , 1, 0)
+
MaskRay wr
MaskRay added inline comments.
Comment at: clang/include/clang/Basic/CodeGenOptions.def:114
+///< Set with -fno-xray-function-index to omit the index section.
+CODEGENOPT(XRayOmitFunctionIndex , 1, 0)
+
Nit: a variable name with a positive meaning may be easier t
This revision was automatically updated to reflect the committed changes.
Closed by commit rG7c7c8e0da4e0: [xray] Option to omit the function index
(authored by ianlevesque).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81995/new/
https://reviews.l
dberris accepted this revision.
dberris added a comment.
This revision is now accepted and ready to land.
LGTM
Thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81995/new/
https://reviews.llvm.org/D81995
ianlevesque created this revision.
ianlevesque added reviewers: dberris, MaskRay, johnislarry.
Herald added subscribers: llvm-commits, Sanitizers, cfe-commits, arphaman,
hiraditya.
Herald added projects: clang, Sanitizers, LLVM.
Add a flag to omit the xray_fn_idx to cut size overhead and relocati