[PATCH] D30642: [ConstantFold] Fix defect in constant folding computation for GEP

[Daniel Berlin via Phabricator via llvm-commits]

dberlin added a comment.

Is there a reason to split NewIdx and Idx at all?
I don't see it in this code and context.

That would solve your problem without introducing yet another array.
It also would be a step on fixing the N^2'dness of this function (since it restarts itself on the changed indices, which is silly)


https://reviews.llvm.org/D30642