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.
Details
- Reviewers
JDevlieghere aprantl Michael137 - Group Reviewers
Restricted Project
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Event Timeline
For watchers -- this is kind of a RFC patch:
- The Dockerfile change will have to be landed separately (first).
- 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 |
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
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