[PATCH] D24480: [InstCombine] remove fold: zext(bool) + C -> bool ? C + 1 : C
Sanjay Patel via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 26 16:11:09 PDT 2016
spatel added a comment.
In https://reviews.llvm.org/D24480#553185, @efriedma wrote:
> LGTM. (If you want to discuss more general questions of when we should prefer select vs. arithmetic, please bring it up on llvmdev.)
Thanks, Eli.
Sure - I'll post the issues raised here on llvm-dev, so we can get more feedback before trying to commit this.
https://reviews.llvm.org/D24480
More information about the llvm-commits
mailing list