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

Javed Absar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 6 07:33:05 PST 2017


javed.absar added a comment.

Yes makes sense if it can be done without breaking some corner case. I will wait a bit for the original author of this algorithm, who I believe are on the review-list, to comment on why originally two arrays, Idxs and NewIdxs, was thought best to have.  
Best Regards
Javed


https://reviews.llvm.org/D30642





More information about the llvm-commits mailing list