[PATCH] D127203: [LoongArch] Add codegen support for the bitwise binary operations and part of other operations

WÁNG Xuěruì via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 16 23:54:58 PDT 2022


xen0n added a comment.

In D127203#3591239 <https://reviews.llvm.org/D127203#3591239>, @SixWeining wrote:

> In D127203#3591233 <https://reviews.llvm.org/D127203#3591233>, @xen0n wrote:
>
>> Thinking about the discrepancy harder, you may want to do an experiment (`movcf2gr` into a register pre-filled with some non-trivial content) to see which version is correct. I don't know if the translators actually *fixed* the original manual during translation (because they have no permission to alter the original document AFAIK).
>
> I alreay did such experiment before removing the `andi`. The actual result is of `movcf2gr` is `GR[rd][0] = CFR[cj] && GR[rd][GRLen-1 .. 1] = 0` and I have confirmed with the author of the original Chinese document that it is realy an ambiguity. He originally wanted to express `GR[rd][0] = CFR[cj],GR[rd][GRLEN-1:1]=0`. This will be fixed in future document updates.

Yeah exactly. We're all fine then ;-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127203



More information about the llvm-commits mailing list