Page MenuHomePhabricator

[OpenMP] Use the OpenMPIRBuilder for "omp parallel"
ClosedPublic

Authored by jdoerfert on Nov 14 2019, 10:34 PM.

Details

Summary

This allows to use the OpenMPIRBuilder for parallel regions. Code was
extracted from D61953 and adapted to work with the new version (D70109).

We only support shared variables for now, that excludes also reductions.

Diff Detail

Event Timeline

jdoerfert created this revision.Nov 14 2019, 10:34 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 14 2019, 10:35 PM
fghanim accepted this revision.Dec 12 2019, 6:07 AM
This revision is now accepted and ready to land.Dec 12 2019, 6:07 AM

Rebase + add a test case

Herald added a project: Restricted Project. · View Herald TranscriptThu, Dec 26, 10:42 PM

Unit tests: fail. 61124 tests passed, 1 failed and 728 were skipped.

failed: Clang.OpenMP/cancel_codegen.cpp

clang-tidy: fail. Please fix clang-tidy findings.

clang-format: fail. Please format your changes with clang-format by running git-clang-format HEAD^ or applying this patch.

Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml

jdoerfert updated this revision to Diff 235449.Fri, Dec 27, 2:24 PM

Rebased on D71948 and combined with OpenMPIRBuilder cancel code gen

Unit tests: unknown.

clang-tidy: unknown.

clang-format: unknown.

Build artifacts: diff.json, console-log.txt

This revision was automatically updated to reflect the committed changes.