[PATCH] D142874: [LoongArch] Implement TargetLowering::isLegalAddressingMode() hook

Mon Jan 30 03:29:25 PST 2023

xen0n added inline comments.

================
Comment at: llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp:3148
+  case 2:
+    // "2*r+r" or "2*r+i" is not allowed.
+    if (AM.HasBaseReg || AM.BaseOffs)
----------------
xen0n wrote:
> Isn't `k*r+r` entirely expressible with `alsl` in case of `k in (2, 4, 8, 16 if LA64)`? You may want to implement it this time, but I'm fine either way.
Hmm, I haven't dug any deeper but if this predicate only governs actions that end up with a `ld/st` then please ignore my comment.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142874