[PATCH] D131380: [LoongArch] Support register-register-addressed GPR loads/stores

Mon Aug 8 05:15:29 PDT 2022

xen0n marked an inline comment as done.
xen0n added inline comments.

================
Comment at: llvm/lib/Target/LoongArch/LoongArchInstrInfo.td:867
+class RegRegLdPat<PatFrag LoadOp, LAInst Inst>
+  : Pat<(i64 (LoadOp (add BaseAddr:$rj, GPR:$rk))),
+        (Inst BaseAddr:$rj, GPR:$rk)>;
----------------
SixWeining wrote:
> Will `f32` and `f64` be coverred in another patch? If so, had better rename the patch title to limit the change to integer load/store.
> Will `f32` and `f64` be coverred in another patch? If so, had better rename the patch title to limit the change to integer load/store.

Yeah I plan to do so; I already mentioned "GPR" in the title. ;-)

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131380