[PATCH] D40721: [InstSimplify] Run constant folding if no other simplifications were possible

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 1 10:45:52 PST 2017


spatel added a comment.

This seems like it would call ConstantFoldXXX a 2nd time for most instructions?

I'd prefer to just add ConstantFoldInsertElementInstruction() to the top of SimplifyInsertElementInst() to keep things symmetrical for now. If this patch makes sense independent of that, then it could be a follow-up?


https://reviews.llvm.org/D40721





More information about the llvm-commits mailing list