[PATCH] D127541: [RISCV] Add cost model for scalable scatter and gather
Fraser Cormack via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 16 02:26:35 PDT 2022
frasercrmck accepted this revision.
frasercrmck added a comment.
LGTM too, with a nit
================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:251
return NumLoads * MemOpCost;
+
}
----------------
Unintentional blank line after `return`?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127541/new/
https://reviews.llvm.org/D127541
More information about the llvm-commits
mailing list