[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