[PATCH] D157416: [RISCV][GlobalISel] Legalize multiplication

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 11 09:22:08 PDT 2023


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/GISel/RISCVLegalizerInfo.cpp:107
+
+    getActionDefinitionsBuilder({G_SMULO, G_UMULO})
+        .widenScalarToNextPow2(0)
----------------
tschuett wrote:
> This builder looks the same with and without the extension? Could you hoist it above the if?
Is the MULO part of this tested?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157416



More information about the llvm-commits mailing list