[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