[PATCH] D86975: [IRSim][IROutliner] Adding the extraction basics for the IROutliner.

[Andrew Litteken via Phabricator via llvm-commits]

AndrewLitteken added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/IROutliner.cpp:238
+  // instruction.
+  for (Instruction &I : *RewrittenBB)
+    if (CallInst *CI = dyn_cast<CallInst>(&I))
----------------
jroelofs wrote:
> Is it guaranteed that we'll find precisely one CallInst in this search? Should there be assertions to that effect?
> 
> Also, is it worth adding a `break` in the inner `if` to early exit once it has been found?
There may be more than one `CallInst` due to lifetime instructions added by the code extractor, but there will not be more than one call to the outlined function.  So a break would be appropriate but an assertion would not be.


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

https://reviews.llvm.org/D86975