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

Quentin Colombet qcolombet at apple.com
Wed Feb 18 09:39:38 PST 2015


> - Now that SimplifyDemandedBits should not simplify multi-used values, is that code still needed?


I believe this is not used anymore.

The select-avx.ll test case that changed that supposed to exerce that path. As you can see, this does not occur :).

> - If it is needed, can you please explain why aren't the uses of TLO.Old being checked (for being VSELECT) and replaced?


Wasn’t it what was happening?
Anyway, no used anymore.

Q.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D6949

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






More information about the llvm-commits mailing list