[PATCH] D61185: [RISCV] Add CostModel tests
Luís Marques via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 23 04:17:38 PDT 2019
luismarques added a comment.
In D61185#1513477 <https://reviews.llvm.org/D61185#1513477>, @asb wrote:
> I think it would make sense to combine the RV32I and RV64I check lines into a shared RVI check-prefix, as I don't think we're expecting different cost model answers for RV32 vs RV64?
That sounds sensible. I did a quick review, to see if there might have been a plausible reason for the split, and I couldn't find any. Will change.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61185/new/
https://reviews.llvm.org/D61185
More information about the llvm-commits
mailing list