[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