[PATCH] D24031: [InstCombine] replace fold of an icmp pattern that should never happen with an assert

Eli Friedman via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 31 15:26:15 PDT 2016


efriedma added a comment.

There isn't any reason to duplicate the combine; instcombine will get to it eventually because it iterates over the function.  I would say just remove the combine and don't add an assertion.


https://reviews.llvm.org/D24031





More information about the llvm-commits mailing list