This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Mark control flow pseudos as convergent
AbandonedPublic

Authored by arsenm on Jul 14 2023, 8:53 AM.

Details

Reviewers
foad
sameerds
rampitec
critson
pravinjagtap
Group Reviewers
Restricted Project

Diff Detail

Event Timeline

arsenm created this revision.Jul 14 2023, 8:53 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 14 2023, 8:53 AM
arsenm requested review of this revision.Jul 14 2023, 8:53 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 14 2023, 8:53 AM
Herald added a subscriber: wdng. · View Herald Transcript
arsenm updated this revision to Diff 540441.Jul 14 2023, 8:56 AM

I'm surprised this did anything, I'm mostly wondering if SI_END_CF needs to be

What does it mean for these control flow pseudos to be convergent? Or is this a simplification to work around other limitations in our current treatment of control flow?

arsenm abandoned this revision.Jul 17 2023, 4:10 AM

I'm starting to form a better idea for how to represent this