[PATCH] D128191: [LoongArch] Combine ((x & shifted_mask) >> shamt) to bstrpick.{w/d}

[WÁNG Xuěruì via Phabricator via llvm-commits]

xen0n added a comment.

Nice overall improvements, thanks!

Only one minor suggestion. Do you think an extra test case that verifies something like `(x & 0xfff0) >> 17` are properly optimized to `0`? Or is such an optimization already covered by all the generic mechanisms so it's not needed here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128191