This is an archive of the discontinued LLVM Phabricator instance.

Fixing test to work when the compiler defaults to a different c++ standard version.
ClosedPublic

Authored by dyung on Dec 6 2016, 3:28 PM.

Details

Summary

This test as originally written fails when run with a compiler that defaults to a different C++ standard. This patch fixes up the test and adds testing it explicitly using c++98 and c++11. The changes were adapted from similar work done in distribute_parallel_for_simd_collapse_messages.cpp.

Diff Detail

Repository
rL LLVM

Event Timeline

dyung updated this revision to Diff 80496.Dec 6 2016, 3:28 PM
dyung retitled this revision from to Fixing test to work when the compiler defaults to a different c++ standard version..
dyung updated this object.
dyung added a reviewer: kkwli0.
dyung added a subscriber: cfe-commits.
kkwli0 edited edge metadata.Dec 6 2016, 3:43 PM

I am okay with the changes. Thanks. Alexey will have the final say.

ABataev accepted this revision.Dec 8 2016, 1:51 AM
ABataev edited edge metadata.

LG

This revision is now accepted and ready to land.Dec 8 2016, 1:51 AM
This revision was automatically updated to reflect the committed changes.