[PATCH] D48491: [X86] Select BEXTR when there is only BMI1.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

Can we DAG combine to X86ISD::BEXTR for these cases? Then we can properly check one use of the inner parts of the pattern. The other option might be to use PatFrags that check for one use like the existing srl_su fragment. My concern is that if the inner nodes of the patterns here have more uses, the emitted pattern won't completely remove the need for the inner node. So you'll end up emitting multiple instructions for this pattern and additional instructions for the remaining use of the inner node.


Repository:
  rL LLVM

https://reviews.llvm.org/D48491