This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Enable more runtime tests and also run them with O3
ClosedPublic

Authored by jdoerfert on Apr 17 2023, 3:55 PM.

Details

Summary

The test run fine on my AMD GPU machine, we should verify them on others
too and put them into our regular testing. Not testing O1/2/3 is really
bad and not testing all architecturs is similarly problematic.

Diff Detail

Event Timeline

jdoerfert created this revision.Apr 17 2023, 3:55 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 17 2023, 3:55 PM
jdoerfert requested review of this revision.Apr 17 2023, 3:55 PM

Works for me on gfx90a

Works for me on gfx90a

Can you verify whether or not __builtin_trap or the s_trap 2 invocation hangs on those boxes? I can't tell if it's just my install or a wider problem.

Works for me on gfx90a

Can you verify whether or not __builtin_trap or the s_trap 2 invocation hangs on those boxes? I can't tell if it's just my install or a wider problem.

I ran the tests several times and it seemed to work. Don't know if that counts exactly as verify.

tianshilei1992 accepted this revision.Jul 24 2023, 9:36 AM
This revision is now accepted and ready to land.Jul 24 2023, 9:36 AM
jhuber6 accepted this revision.Jul 24 2023, 9:36 AM

Sure, a few tests already hang for me on AMDGPU for unknown reasons, so the extra one w/ __builtin_trap is the least of my worries.

Herald added a project: Restricted Project. · View Herald TranscriptJul 31 2023, 3:55 PM