[PATCH] D147406: [LoongArch] Optimize multiplication with immediates

Ben Shi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 2 01:52:04 PDT 2023


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


================
Comment at: llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp:3143
+        ((Imm - 2).isPowerOf2() || (Imm - 4).isPowerOf2() ||
+         (Imm - 4).isPowerOf2() || (Imm - 8).isPowerOf2()))
+      return true;
----------------
xen0n wrote:
> Do you mean `Imm - 8` and `Imm - 16` instead?
Yes. Sorry for such a mistake. Has fixed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147406



More information about the llvm-commits mailing list