[PATCH] D79483: [CostModel] Replace getUserCost with getInstructionCost.
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 18 03:20:00 PDT 2022
RKSimon added inline comments.
================
Comment at: llvm/test/Analysis/CostModel/AArch64/sve-math.ll:15
; LATE-LABEL: 'fadd_v2f64'
-; LATE-NEXT: Cost Model: Found an estimated cost of 3 for instruction: %r = fadd <vscale x 2 x double> %a, %b
+; LATE-NEXT: Cost Model: Found an estimated cost of 1 for instruction: %r = fadd <vscale x 2 x double> %a, %b
; LATE-NEXT: Cost Model: Found an estimated cost of 1 for instruction: ret <vscale x 2 x double> %r
----------------
@samparker Just to be clear - you're happy for me to make the latency=3 default x86-only?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79483/new/
https://reviews.llvm.org/D79483
More information about the llvm-commits
mailing list