[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