[PATCH] D72873: [clang][xray] Add -fxray-ignore-loops option

[Aditya Kumar via Phabricator via cfe-commits]

hiraditya added inline comments.


================
Comment at: clang/lib/CodeGen/CodeGenFunction.cpp:821
             llvm::itostr(CGM.getCodeGenOpts().XRayInstructionThreshold));
+      if (CGM.getCodeGenOpts().XRayIgnoreLoops) {
+        Fn->addFnAttr("xray-ignore-loops");
----------------
we can remove braces here.


================
Comment at: clang/test/CodeGen/xray-ignore-loops.cpp:5
+  return 1;
+}
+
----------------
I think we need one more test case of a function having a loop.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72873/new/

https://reviews.llvm.org/D72873