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

Quentin Colombet qcolombet at apple.com
Wed Jan 14 13:47:32 PST 2015


Hi Gil,

How does this impact the performances on the whole test suite + externals?

Indeed, when I did r221429, handling the multiple uses like you do in the proposal here, lead to bad regressions. That said, what I tried at that time was slightly different so it may not be the case here.
Anyway, we need to ascertain that.

Let me know if you do not have access to externals, I may be able to run some benchmarks on your behalf.

Thanks,
-Quentin


http://reviews.llvm.org/D6949

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






More information about the llvm-commits mailing list