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

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 4 02:58:27 PST 2019


lebedev.ri added a comment.

In D56715#1382867 <https://reviews.llvm.org/D56715#1382867>, @RKSimon wrote:

> How come SimplifyDemandedBits isn't doing this for us?


Note that we are in X86ISelDAGToDAG. I don't observe anything SimplifyDemandedBits-related here.
There is SimplifyDemandedBits stuff in X86TargetLowering, but is that not run before we end up here?
Or am i missing some path how SimplifyDemandedBits will be run after `X86DAGToDAGISel::matchBitExtract()`?
If i am, i'd love to learn about it, and use it to solve this.


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