[PATCH] D67875: [X86] X86DAGToDAGISel::matchBEXTRFromAndImm(): if can't use BEXTR, fallback to BZHI (PR43381)

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

In D67875#1678334 <https://reviews.llvm.org/D67875#1678334>, @craig.topper wrote:

> What if we just do the larger than 32-bit mask? Its not clear that making BZHI just to fold a load is an improvement. You have to materialize an immediate instead so the total uops increased.


Actually, i think it's clearly still not an improvement: https://godbolt.org/z/1M1deG


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67875