[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