This is an archive of the discontinued LLVM Phabricator instance.

[NFC][AArch64] Add cost model tests for extending loads
ClosedPublic

Authored by david-arm on Apr 12 2023, 6:14 AM.

Diff Detail

Event Timeline

david-arm created this revision.Apr 12 2023, 6:14 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 12 2023, 6:14 AM
david-arm requested review of this revision.Apr 12 2023, 6:14 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 12 2023, 6:14 AM
david-arm retitled this revision from [AArch64][CostModel] Make sext/zext free if folded into a masked load to [NFC][LoopVectorize][AArch64] Add cost model tests for extending loads.Apr 12 2023, 6:15 AM
david-arm edited the summary of this revision. (Show Details)
david-arm retitled this revision from [NFC][LoopVectorize][AArch64] Add cost model tests for extending loads to [NFC][AArch64] Add cost model tests for extending loads.Apr 12 2023, 6:15 AM
david-arm updated this revision to Diff 514230.Apr 17 2023, 7:50 AM
  • Added test cases for when loading into a legal type and extending it.
MattDevereau accepted this revision.Apr 18 2023, 6:05 AM
MattDevereau added inline comments.
llvm/test/Analysis/CostModel/AArch64/masked_ldst.ll
41

It's a bit weird that this file is called masked_ldst.ll when it doesn't have a single store in it, I'd consider renaming it

This revision is now accepted and ready to land.Apr 18 2023, 6:05 AM
This revision was landed with ongoing or failed builds.Apr 19 2023, 8:44 AM
This revision was automatically updated to reflect the committed changes.