[PATCH] Removed pattern checks in instruction simplify/ combine which are now handled in SimplifyUsingDistributiveLaws()

Jingyue Wu jingyue at google.com
Tue Jun 24 22:06:34 PDT 2014


Looks good in general. I am reading this patch out of context. Are you removing these because you added similar logic in InstCombine? A further question I am curious about is what's the split between InstructionSimplify and InstCombine. Both seem to simplify instructions. What should be in one versus the other?

http://reviews.llvm.org/D4253






More information about the llvm-commits mailing list