This is an archive of the discontinued LLVM Phabricator instance.

[libc++] CI: Run -std=c++03 on Clang ToT
ClosedPublic

Authored by ldionne on Jul 15 2021, 3:06 PM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Commits
rG45478619e3eb: [libc++] CI: Run -std=c++03 on Clang ToT

Diff Detail

Event Timeline

ldionne created this revision.Jul 15 2021, 3:06 PM
ldionne requested review of this revision.Jul 15 2021, 3:06 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 15 2021, 3:06 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne updated this revision to Diff 359794.Jul 19 2021, 8:06 AM

Rebase to trigger CI.

ldionne updated this revision to Diff 359806.Jul 19 2021, 8:31 AM

Print the compiler version for debugging

ldionne updated this revision to Diff 359958.Jul 19 2021, 4:38 PM

Poke CI again -- did the nodes update their Clang ToT yet?

ldionne updated this revision to Diff 360416.Jul 21 2021, 6:22 AM

Poke CI: did the bots update to ToT?

ldionne updated this revision to Diff 361681.Jul 26 2021, 8:58 AM

Poke CI - I think the bots should have updated now.

@mizvekov For some reason, it looks like the tests are still failing with Clang ToT, even with all your patches applied. Any idea?

We need to fix this for the LLVM 13 release, or users will be unable to compile in C++03 mode (cause libc++ won't work).

@mizvekov For some reason, it looks like the tests are still failing with Clang ToT, even with all your patches applied. Any idea?

We need to fix this for the LLVM 13 release, or users will be unable to compile in C++03 mode (cause libc++ won't work).

It's working on compiler explorer / clang trunk: https://godbolt.org/z/E6xP4qfPa

Maybe that bot has very outdated clang?

@mizvekov For some reason, it looks like the tests are still failing with Clang ToT, even with all your patches applied. Any idea?

We need to fix this for the LLVM 13 release, or users will be unable to compile in C++03 mode (cause libc++ won't work).

It's working on compiler explorer / clang trunk: https://godbolt.org/z/E6xP4qfPa

Maybe that bot has very outdated clang?

Hmm, interesting, thanks for checking. I think the bots should be updating their image automatically, but perhaps that doesn't work as intended. I'm trying to make them refresh their Docker image here: 496a3815f43d20853a4fad77c7d70414385c3b06

ldionne accepted this revision.Jul 27 2021, 11:12 AM

Okay, looks like I guessed correctly. The Docker images were being cached and had not updated for a while. Changing the Dockerfile forced them to rebuild and the C++03 job passed. I'll ship this now.

This revision is now accepted and ready to land.Jul 27 2021, 11:12 AM
This revision was landed with ongoing or failed builds.Jul 27 2021, 11:13 AM
This revision was automatically updated to reflect the committed changes.