[PATCH] D47874: [SCEVExp] Advance found insertion point until we find a non-dbg instruction.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: lib/Analysis/ScalarEvolutionExpander.cpp:105
   }
 
   return IP;
----------------
vsk wrote:
> Why not skip debug instructions here, instead? It should make less work for callers of findInsertPointAfter, right?
IIRC Eli suggested to not do it in this function, as it should behave similar to BasicBlock::getFirstInsertionPt , which also does not skip debug info.


https://reviews.llvm.org/D47874