[PATCH] D122336: [InstrProfiling] No runtime hook for unused funcs

Petr Hosek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 23 23:57:12 PDT 2022


phosek accepted this revision.
phosek added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Transforms/Instrumentation/InstrProfiling.cpp:569-573
+    if (!CoverageNamesVar) {
+      return MadeChange;
+    } else if (!NeedsRuntimeHook) {
+      return MadeChange;
+    }
----------------
This could be combined into a single statement.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122336



More information about the cfe-commits mailing list