Page MenuHomePhabricator

[AMDGPU] pin lit test divergent-unswitch.ll to the old pass manager
ClosedPublic

Authored by sameerds on Jan 20 2021, 8:09 AM.

Details

Summary

The loop-unswitch transform should not be performed on a loop whose
condition is divergent. For this to happen correctly, divergence
analysis must be available. The existing divergence analysis has not
been ported to the new pass manager yet. As a result, loop unswitching
on the new pass manager is currently unsafe on targets that care about
divergence.

This test is temporarily disabled to unblock work on the new pass
manager. The issue is now tracked in bug 48819.

Change-Id: I05da823b1377c9a1d61ce8bc8ea11eb0bb09f0f1

Diff Detail

Event Timeline

sameerds created this revision.Jan 20 2021, 8:09 AM
sameerds requested review of this revision.Jan 20 2021, 8:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 20 2021, 8:09 AM
sameerds added a subscriber: aeubanks.
foad accepted this revision.Jan 20 2021, 8:16 AM

LGTM.

This revision is now accepted and ready to land.Jan 20 2021, 8:16 AM
This revision was landed with ongoing or failed builds.Jan 20 2021, 8:33 AM
This revision was automatically updated to reflect the committed changes.