[all-commits] [llvm/llvm-project] 2e0bf6: [LV][AArch64] Fix reductions costs in strict-fadd-...
David Green via All-commits
all-commits at lists.llvm.org
Thu Jul 20 02:34:20 PDT 2023
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 2e0bf67df1437cb0156d7f5dd9e1b701749f96ca
https://github.com/llvm/llvm-project/commit/2e0bf67df1437cb0156d7f5dd9e1b701749f96ca
Author: David Green <david.green at arm.com>
Date: 2023-07-20 (Thu, 20 Jul 2023)
Changed paths:
M llvm/test/Transforms/LoopVectorize/AArch64/strict-fadd-cost.ll
Log Message:
-----------
[LV][AArch64] Fix reductions costs in strict-fadd-cost.ll. NFC
These tests were originally added in 0aff1798b5721d5f95d16f465b99d, where they
were measuring the cost of fadd and fmuladd reductions, which should be fairly
high cost. For some reason, due to the forced vector factors, the debug costs
of each instruction are printed twice by the vectorizer. Once as if the
instruction is a simple fadd/fmuladd, and later with the correct reduction
cost.
In d827865e9f778f5b27edb2afe003c2a the costs were updated to match the first
print statements, where they would be better to match the second to test the
cost of the reduction.
This patch returns them to testing the original reduction costs.
More information about the All-commits
mailing list