This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Enable more tests
AbandonedPublic

Authored by jdoerfert on Mar 26 2023, 4:09 PM.

Details

Summary

This enables tests to run on AMDGPU and also enables optimization for the compilation.

Diff Detail

Event Timeline

jdoerfert created this revision.Mar 26 2023, 4:09 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 26 2023, 4:09 PM
jdoerfert requested review of this revision.Mar 26 2023, 4:09 PM

They all pass for me on an X86 + gfx90a, can people try them on different setups and report results here, plz.

jdoerfert retitled this revision from [OpenMP] Enable more tests (for AMDGPU) to [OpenMP] Enable more tests.Mar 26 2023, 4:11 PM
jdoerfert edited the summary of this revision. (Show Details)
jdoerfert updated this revision to Diff 508446.Mar 26 2023, 4:13 PM

@doru1004 is interested in these tests being enabled.
thx for adding them.

@doru1004 is interested in these tests being enabled.
thx for adding them.

@ronlieb Can you test this commit on the builedbot too?

ronlieb accepted this revision.Mar 26 2023, 5:07 PM

i applied the patch to a gfx908 and a gfx906 system.
tests all passed , and the gfx906 should be the same as the amdgpu buildbot.

one odd thing i ran into was that the raw patch did not apply to
openmp/libomptarget/test/offloading/barrier_fence.c

perhaps it just needs rebase ?

any way, seems like its worth landing after you get some feedback from other platforms.
or if none seems forthcoming ,land it. so i gave it the old LGTM

This revision is now accepted and ready to land.Mar 26 2023, 5:07 PM

one odd thing i ran into was that the raw patch did not apply to
openmp/libomptarget/test/offloading/barrier_fence.c

That one is included in the fence patch. I will ensure it works once I upstream.

jdoerfert abandoned this revision.Apr 17 2023, 3:58 PM

Subsumed by D148576