[PATCH] Revert 221429 and fix bug in TargetLowering::SimplifyDemandedBits

Gil Rapaport gil.rapaport at intel.com
Thu Jan 29 06:09:20 PST 2015


Hi Quentin,

Ahh, right, removing the SHRUNK_BLEND node would re-invalidate the booleaness property of the VBLEND condition.
I'll prepare a new patch which keeps the SHRUNK_BLEND node and only fixes the DemandedMask bug (to protect other callers to SimplifyDemandedBits). I'll also look into narrowing this optimization to skip cases where the sext-in-reg could provably be removed on account of all bits already being set.

Thanks,
Gil


http://reviews.llvm.org/D6949

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list