[PATCH] D80975: [SCEV][IndVarSimplify] insert point should not be block front if the front instruction is a PHI

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 2 01:04:57 PDT 2020


lebedev.ri added a comment.

Naively i would think that ScalarEvolutionExpander should not second-guess/re-guess specified insertion point,
and the callers (possibly ScalarEvolutionExpander itself too) should be adjusted to specify `getFirstInsertionPt()` as the insertion point.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80975





More information about the llvm-commits mailing list