[PATCH] D51584: [IndVars] Smart hard uses detection

[Changpeng Fang via Phabricator via llvm-commits]

cfang added a comment.

I am working on a regression caused by this patch.  It is a memory access fault actually.
However, if we put back the original consition, my test would pass.

Could you explain why the original condition was removed in your patch?

+if (ExitValue->getSCEVType()>=scMulExpr) {

  if (!isa<SCEVConstant>(ExitValue) && hasHardUserWithinLoop(L, Inst))
    continue;

+  }


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

https://reviews.llvm.org/D51584