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

[Lu Weining via Phabricator via llvm-commits]

SixWeining added inline comments.


================
Comment at: llvm/lib/Target/LoongArch/LoongArchInstrInfo.td:578
+def : PatGprImm<shl, SLLI_W, uimm5>;
+def : PatGprImm<shl, SRAI_W, uimm5>;
+def : PatGprImm<shl, SRLI_W, uimm5>;
----------------
Typo. Should be `sra`.


================
Comment at: llvm/lib/Target/LoongArch/LoongArchInstrInfo.td:579
+def : PatGprImm<shl, SRAI_W, uimm5>;
+def : PatGprImm<shl, SRLI_W, uimm5>;
+} // Predicates = [IsLA32]
----------------
Ditto. Should be `srl`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127203