[PATCH] D24480: [InstCombine] remove fold: zext(bool) + C -> bool ? C + 1 : C

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 12 17:26:08 PDT 2016


arsenm added a subscriber: arsenm.
arsenm added a comment.

In https://reviews.llvm.org/D24480#540557, @majnemer wrote:

> I think IR should be canonicalized with ValueTracking in mind (ComputeKnownBits, etc.)
>  That SelectionDAG doesn't know how to make selects lower nicely is a problem but it can be localized late in codegen: either in SDAG itself or CGP.


+1


https://reviews.llvm.org/D24480





More information about the llvm-commits mailing list