[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