This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Extend and reorganize memory legalizer tests
AbandonedPublic

Authored by scott.linder on Nov 17 2020, 4:23 PM.

Details

Reviewers
None
Summary
  • Rename some tests to try to make a convention (where all components are optional) of:

    <addrspace>_<syncscope>_<memory-orders>_<operation>
  • Split up at a level of granularity appropriate for the different RUN lines (i.e. split on addrspace so GFX6 can avoid FLAT) and that makes running a specific test reasonable in terms of wall time taken. This also means when run as part of the test suite the testing is not one serial bottleneck.
  • Auto-generate check lines with update_llc_test_checks.py to make future maintenance more tractable.

Diff Detail

Event Timeline

scott.linder created this revision.Nov 17 2020, 4:23 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 17 2020, 4:23 PM
scott.linder requested review of this revision.Nov 17 2020, 4:23 PM
scott.linder abandoned this revision.Nov 17 2020, 4:26 PM

Opened by mistake, intended to update D91545