[PATCH] D129357: [LoongArch] Implement OR combination to generate bstrins.w/d

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

xen0n added a comment.

In D129357#3642259 <https://reviews.llvm.org/D129357#3642259>, @SixWeining wrote:

> In D129357#3638731 <https://reviews.llvm.org/D129357#3638731>, @xen0n wrote:
>
>> What about the arguably more common case of `a = b | ((c & mask) << shamt)` (when `c` is not known to be small enough to not require the masking) or `a = b | (c & mask)`, where all positions in `a` to be overwritten by the incoming bits are known to be zero?
>>
>> One of the applicable real-world scenario is RISC code generation, where a lot of ORing between a "clean" insn template and shifted/masked operands take place. One doesn't mask the bits in the template hence the optimization chance is missed.
>>
>> The other cases LGTM, though.
>
> Sorry to reply so late. Good suggestion! And `a = b | ((c << shamt) & shifted_mask)` seems also apply. Let me try to update the revision. Thanks!

Yeah of course. Thanks for considering the suggestion!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129357