This is an archive of the discontinued LLVM Phabricator instance.

[llvm][openacc] Fix OpenACC frontend test
ClosedPublic

Authored by razvanlupusoru on Jun 7 2023, 2:42 PM.

Details

Summary

After D152389, single reduction clause constraint was removed.
This broke the OpenACC frontend test which seems to rely on specific
allowed clause ordering.

Diff Detail

Event Timeline

razvanlupusoru created this revision.Jun 7 2023, 2:42 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 7 2023, 2:42 PM
Herald added a subscriber: sunshaoce. · View Herald Transcript
razvanlupusoru requested review of this revision.Jun 7 2023, 2:42 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 7 2023, 2:42 PM
vzakhari accepted this revision.Jun 7 2023, 2:44 PM
This revision is now accepted and ready to land.Jun 7 2023, 2:44 PM

I will land this soon - I am just waiting for my local build to complete and confirm one final time that this test is indeed fully resolved.

Updated serial clause order.

This revision was landed with ongoing or failed builds.Jun 7 2023, 3:14 PM
This revision was automatically updated to reflect the committed changes.