[PATCH] D36055: [X86] Disable creating BEXTR from shift and mask operations with BMI. Only do it for TBM.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a comment.

In https://reviews.llvm.org/D36055#826654, @craig.topper wrote:

> If we create X86ISD::BEXTRI with split immediates. How would we create the combined immediate to select BEXTRI instruction. Maybe a custom inserter?


Not sure - I'm struggling to think of something that will deal with all the useful cases where we should still try to use BEXTR (foldable loads on BMI + multiple uses on BMI/TBM).


https://reviews.llvm.org/D36055