This is an archive of the discontinued LLVM Phabricator instance.

[DO NOT COMMIT] Test if the libcxx CI system uses the updated clang compiler
AbandonedPublic

Authored by ChuanqiXu on Nov 3 2021, 11:50 PM.

Details

Reviewers
None
Summary

The content of this diff is the same as D109433. The CI's result of D109433 would fail all the way. It runs well locally. And from the error message from the CI, the reason looks like it didn't use the updated compiler. The intention of this diff is to test this.

Diff Detail

Event Timeline

ChuanqiXu created this revision.Nov 3 2021, 11:50 PM
ChuanqiXu requested review of this revision.Nov 3 2021, 11:50 PM
ChuanqiXu removed subscribers: mgorny, arichardson, lxfind.
ChuanqiXu added a project: Restricted Project.Nov 4 2021, 12:21 AM
ChuanqiXu abandoned this revision.Nov 4 2021, 12:42 AM
ChuanqiXu updated this revision to Diff 384971.Nov 4 2021, 10:29 PM

Emit an error if it find the compiler didn't implement __cpp_impl_coroutine macro. It should be implemented in ec11715.

ChuanqiXu added inline comments.Nov 4 2021, 11:44 PM
libcxx/include/coroutine
43–45

I added a check here to see if the compiler introduced macro __cpp_impl_coroutine, which should be introduced at https://github.com/llvm/llvm-project/commit/ec117158a390a0ebf64377caa5abd0c976df8f7a.

ChuanqiXu abandoned this revision.Nov 5 2021, 10:23 PM
ChuanqiXu updated this revision to Diff 385395.Nov 7 2021, 9:14 PM

Update again

ChuanqiXu updated this revision to Diff 385397.Nov 7 2021, 9:17 PM

Test again

ChuanqiXu abandoned this revision.Nov 8 2021, 6:02 PM
libcxx/test/std/language.support/support.limits/support.limits.general/version.version.pass.cpp