This is an archive of the discontinued LLVM Phabricator instance.

[libc++][ci] Re-enable the bootstrapping build
ClosedPublic

Authored by ldionne on Jan 24 2022, 12:50 PM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Commits
rGc7b255e5a8ed: [libc++][ci] Re-enable the bootstrapping build

Diff Detail

Event Timeline

ldionne created this revision.Jan 24 2022, 12:50 PM
ldionne requested review of this revision.Jan 24 2022, 12:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 24 2022, 12:50 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript

@saugustine This started failing recently and the cause isn't obvious to me -- would you mind taking a look? The issue reproduces locally with

./libcxx/utils/ci/run-buildbot-container
./libcxx/utils/ci/run-buildbot bootstrapping-build # run this from the container

Gentle ping on this. I'll have to disable the GDB pretty-printers test and re-enable the bootstrapping build configuration if this isn't addressed, cause I really want the bootstrapping build to be tested in CI again by the time we release LLVM 14.

ldionne updated this revision to Diff 404901.Feb 1 2022, 5:49 AM

Mark test as unsupported on Clang 14 for now.

ldionne accepted this revision.Feb 1 2022, 12:28 PM
This revision is now accepted and ready to land.Feb 1 2022, 12:28 PM
This revision was automatically updated to reflect the committed changes.

Hi Louis,

My apologies for missing this when it was sent.

The problem here is that your docker image uses gdb 9.2, which doesn't handle dwarf5 info as generated by clang. You can read some discussion about this here:

https://discourse.llvm.org/t/gdb-10-1-cant-read-clangs-dwarf-v5/6035/11

The resolution of that discussion is that the package maintainer should be sure that clang defaults to a format the system debugger is capable of reading.

So you have two options: Make this build of clang default to dwarf4, or upgrade the docker installed copy of gdb to 10.2 or greater.

Not sure which is the best one for you. But in general this test works fine with newer copies of gdb and newer copies of clang, and disabling it probably isn't quite the right thing to do.

Herald added a project: Restricted Project. · View Herald TranscriptApr 14 2022, 4:58 PM
jgorbe added a subscriber: jgorbe.Apr 14 2022, 5:01 PM