This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] update test cases for -std=c++11 compile
ClosedPublic

Authored by kkwli0 on Jul 15 2016, 10:59 AM.

Details

Summary

target_parallel_for_simd_collapse_messages.cpp and target_parallel_for_simd_ordered_messages.cpp give different diagnostic messages in compiling with -std=c++11. The test cases are updated to make it compatible.

Diff Detail

Repository
rL LLVM

Event Timeline

kkwli0 updated this revision to Diff 64167.Jul 15 2016, 10:59 AM
kkwli0 retitled this revision from to [OpenMP] update test cases for -std=c++11 compile.
kkwli0 updated this object.
kkwli0 added reviewers: ABataev, probinson.
kkwli0 updated this revision to Diff 64270.Jul 17 2016, 7:31 PM

Add more test paths.

probinson edited edge metadata.Jul 17 2016, 7:44 PM

I can confirm that these pass in our environment--thanks!
Somebody familiar with OpenMP should give final approval though.

BTW normally you want to add cfe-commits as a subscriber but it looks like you missed that?

Add cfe-commits

ABataev accepted this revision.Jul 17 2016, 8:16 PM
ABataev edited edge metadata.

LG

This revision is now accepted and ready to land.Jul 17 2016, 8:16 PM
This revision was automatically updated to reflect the committed changes.