This is an archive of the discontinued LLVM Phabricator instance.

[AArch64] Add cost tests for fast fadd reduction
AbandonedPublic

Authored by virnarula on Jun 10 2022, 3:54 PM.

Details

Summary

Add cost model tests for upcoming dot product lowering change. Related because we had to fix the arm64 cost model for our change.

Diff Detail

Event Timeline

virnarula created this revision.Jun 10 2022, 3:54 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 10 2022, 3:54 PM
Herald added a subscriber: tschuett. · View Herald Transcript
virnarula requested review of this revision.Jun 10 2022, 3:54 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 10 2022, 3:54 PM
virnarula retitled this revision from [Matrix] Add cost tests for reduction add and fast fadd to [Matrix] Add cost tests for fast fadd reduction.Jun 10 2022, 3:57 PM
virnarula edited the summary of this revision. (Show Details)
virnarula added reviewers: fhahn, thegameg, anemet.
fhahn added a comment.Jun 13 2022, 9:37 AM

Thanks! Could you also add tests for reductions with just the reassoicate fast-math flag?

Also, this is not really specific to matrix support in LLVM, but applies generally to the AArch64 cost modeling. So it would be better to use [AArch64] in the description.

llvm/test/Analysis/CostModel/AArch64/reduce-fadd.ll
25

I think the strict part here refers to the ordering requirement, i.e. strict sequential order.

So it would probably be better to name the function @fast_fp_reductions.

40

can you also throw in some vectors with some odd numbers of elements.

virnarula retitled this revision from [Matrix] Add cost tests for fast fadd reduction to [AArch64] Add cost tests for fast fadd reduction.Jun 13 2022, 9:41 AM
virnarula abandoned this revision.Jun 13 2022, 3:03 PM

Replaced with the following review: https://reviews.llvm.org/D127682. Squashing prior to running arc diff created a new review instead of updating this one.