[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