[PATCH] D56715: X86DAGToDAGISel::matchBitExtract(): prepare 'control' in 32 bits

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 4 10:37:03 PST 2019


craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.

I don't think SimplifyDemandedBits can do anything for this. We're taking an i8 shift value and turning it into a control. The issue is entirely within how this code promoted that i8 value to match the width of the BEXTR/BZHI.

LGTM


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56715/new/

https://reviews.llvm.org/D56715





More information about the llvm-commits mailing list