[PATCH] D55187: Set norecurse attribute on functions that have debug infos.

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc requested changes to this revision.
chandlerc added inline comments.
This revision now requires changes to proceed.


================
Comment at: lib/Transforms/IPO/FunctionAttrs.cpp:1311
   // marked norecurse, so any called from F to F will not be marked norecurse.
   for (Instruction &I : instructions(*F))
+    if (!isa<DbgInfoIntrinsic>(I))
----------------
aprantl wrote:
> Is there something like on the function level?
> 
> http://www.llvm.org/doxygen/classllvm_1_1BasicBlock.html#a126c95a0ff18e58ac0c2c7439fb2dc6d
Yeah, I'd much prefer to iterate the filtered sequence here.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55187