[PATCH] D133552: [RISCV] Add cost model for integer and float vector arithmetic instructions.

[Philip Reames via Phabricator via llvm-commits]

reames added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:918
+    return BaseT::getArithmeticInstrCost(Opcode, Ty, CostKind, Op1Info,
+                                         Op2Info);
+  }
----------------
You have a bug here - you didn't pass through the last two parameters and thus get the defaults when calling the base class version of the function.

I found this when iterating on a local version of this patch with the intention of subsetting and giving feedback on the remaining part. I went ahead and landed the plumbing for this change as d2cf0bd7 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133552/new/

https://reviews.llvm.org/D133552