This is an archive of the discontinued LLVM Phabricator instance.

[OPENMP] Add support for 'atomic_default_mem_order' clause on requires directive
ClosedPublic

Authored by patricklyster on Oct 22 2018, 10:28 AM.

Details

Summary

Added new atomic_default_mem_order clause with seq_cst, acq_rel and relaxed modifiers to OMP5.0 requires directive.

Renamed test files relating to requires tests.

Diff Detail

Event Timeline

patricklyster created this revision.Oct 22 2018, 10:28 AM

Check the code formatting, please. There should be lines longer than 80 symbols.

clang/include/clang/AST/OpenMPClause.h
886

Check formatting here

906

Also, formatting

930

Again, probably too long line.

patricklyster added inline comments.Oct 22 2018, 12:14 PM
clang/include/clang/AST/OpenMPClause.h
930

None of these lines are longer than 80 symbols - hard to see this on Phabricator though. All this code was formatted using the clang-format tool. Would you still like me to change it?

This revision is now accepted and ready to land.Oct 22 2018, 12:36 PM
clang/test/OpenMP/requires_unified_address_ast_print.cpp