Page MenuHomePhabricator

Fix for https://bugs.llvm.org/show_bug.cgi?id=39970
ClosedPublic

Authored by AndreyChurbanov on Dec 12 2018, 5:40 AM.

Details

Summary

Some tests require more than 1 thread to pass. The patch checks max threads, and if iti is 1 then sets number of threads to more than one.

Exceptions:
ompt/misc/api_calls_places.c - broke the standard requirement, fixed this;
worksharing/for/omp_doacross.c - could not reproduce the failure, increased the number of threads just in case.

Diff Detail

Repository
rL LLVM

Event Timeline

AndreyChurbanov accepted this revision.Dec 13 2018, 2:02 AM

Tom Stellard confirmed that this patch fixes all the issues reported. So, once this patch does not touch source code, I will commit it without formal review.

This revision is now accepted and ready to land.Dec 13 2018, 2:02 AM
This revision was automatically updated to reflect the committed changes.