This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add test cases for cost modelling more shuffle kinds
ClosedPublic

Authored by luke on Mar 23 2023, 10:57 AM.

Details

Summary

Upcoming changes to how we cost model these shuffles will end up
affecting reductions in the SLP vectorizer, so two cases have been added
to illustrate this

Diff Detail

Event Timeline

luke created this revision.Mar 23 2023, 10:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 23 2023, 10:57 AM
luke requested review of this revision.Mar 23 2023, 10:57 AM
reames accepted this revision.Mar 23 2023, 2:01 PM

LGTM - though can you split this into two patches when you land? The SLP bit deserves a slightly different description focus.

This revision is now accepted and ready to land.Mar 23 2023, 2:01 PM
luke updated this revision to Diff 508075.Mar 24 2023, 6:48 AM

Split up into several smaller commits

  • [RISCV] Add test cases for modeling more shuffle kinds
  • [RISCV] Enable SLP in RISC-V SLP reduction tests
  • [RISCV] Add test case for two equivalent reductions