[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