[PATCH] D48105: [llvm][Instrumentation] Add Call Graph Profile pass

Michael Spencer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 13 11:48:46 PDT 2018


Bigcheese added inline comments.


================
Comment at: lib/Transforms/Instrumentation/CGProfile.cpp:48
+
+  for (auto &F : M) {
+    if (F.isDeclaration())
----------------
compnerd wrote:
> Can this not be `const auto &F : M`?
No, because of getAnalysis.


https://reviews.llvm.org/D48105





More information about the llvm-commits mailing list