[PATCH] D76124: [TTI] Remove getOperationCost

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 13 05:06:51 PDT 2020


samparker marked an inline comment as done.
samparker added inline comments.


================
Comment at: llvm/test/Analysis/CostModel/X86/costmodel.ll:11
 
 define i64 @foo(i64 %arg) {
 ; LATENCY-LABEL: 'foo'
----------------
lebedev.ri wrote:
> Changes here look wrong to me
I was suspicious, but since these should now be the costs reported by the backend so I'd like to get some clarification. I'm assuming it's instead just returning a default cost.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76124





More information about the llvm-commits mailing list