This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Add a CI job for the LLDB data formatters
AbandonedPublic

Authored by ldionne on Sep 9 2022, 4:14 PM.

Details

Reviewers
JDevlieghere
aprantl
Michael137
Group Reviewers
Restricted Project
Summary

This patch adds a CI job running the LLDB data formatters in the libc++
CI. This should avoid breaking the LLDB data formatters when e.g. the
layout of libc++ data structures is changed.

Diff Detail

Event Timeline

ldionne created this revision.Sep 9 2022, 4:14 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 9 2022, 4:14 PM
Herald added a subscriber: arichardson. · View Herald Transcript
ldionne requested review of this revision.Sep 9 2022, 4:14 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 9 2022, 4:14 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript

For watchers -- this is kind of a RFC patch:

  1. The Dockerfile change will have to be landed separately (first).
  2. I'd like to quantify how slow this job is before we add it. We should avoid adding a lot of latency to the libc++ CI pipeline.

Also, this is not a hard commitment to start maintaining LLDB's data formatters. I spoke to @aprantl and @JDevlieghere and we agreed that this should be used as a way to avoid disruption on both the libc++ and the LLDB side. If we run into LLDB data formatters issues with a libc++ patch, we should work with the LLDB folks (and they agreed to be responsive) to resolve the issue and unblock the libc++ patch. That way, everyone's happy. Let's just hope the turnaround time for this job isn't prohibitive.

Thanks a lot for working on this!

libcxx/utils/ci/run-buildbot
475

When this is too slow we could consider merging it with the bootstrapping build. They have quite a bit of overlap.

That isn't a great solution, but it may help us. I didn't find a way a in Buildkite to re-use the build from an other step as basis for a new step

ldionne updated this revision to Diff 459621.Sep 12 2022, 6:45 PM

Rebase onto main -- this should run the new CI job now.

Thanks for working on this, Louis. Looks good from the LLDB perspective.

ldionne abandoned this revision.Sep 1 2023, 7:47 AM

Since I probably won't have time to pick this up and finish it this week, I am closing this and I moved it over to GitHub here: https://github.com/llvm/llvm-project/pull/65174