[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