This is an archive of the discontinued LLVM Phabricator instance.

new worker and builder for clangd
ClosedPublic

Authored by kuhnel on Nov 9 2020, 11:48 PM.

Details

Summary

Adding new worker and builder to cover remote indexing feature of clangd.

Diff Detail

Event Timeline

kuhnel created this revision.Nov 9 2020, 11:48 PM
kuhnel requested review of this revision.Nov 9 2020, 11:48 PM
kuhnel edited the summary of this revision. (Show Details)Nov 9 2020, 11:49 PM
kuhnel added reviewers: sammccall, kadircet, kbobyrev.
kbobyrev added inline comments.Nov 10 2020, 12:07 AM
buildbot/osuosl/master/config/workers.py
210

nit: typo here and above in "Ubuntu"

kuhnel updated this revision to Diff 304065.Nov 10 2020, 12:16 AM

fixed typos

kuhnel marked an inline comment as done.Nov 10 2020, 12:16 AM

Thanks so much for doing this!

buildbot/osuosl/master/config/builders.py
1875

nit: tsan or TSan (though the builders seem to be conventionally all-lowercase)

1879

can we add checks=['check-clangd']?

As well as being faster, building/testing clangd without building clang is something we want to work. (clangd doesn't have a dependency on all of clang).
See D90518 where this was broken for months and we didn't know.

1882

Don't we need -DCMAKE_CXX_COMPILER=clang++ or clang++-11 or so? (and ditto -DCMAKE_C_COMPILER)

kuhnel updated this revision to Diff 304083.Nov 10 2020, 1:21 AM
kuhnel marked 3 inline comments as done.

addressed Sam's comments

kuhnel added inline comments.Nov 10 2020, 1:24 AM
buildbot/osuosl/master/config/builders.py
1882

You're right, CMake was using GCC. Let me set the environment variable in the Docker container, as we're configuring the other build tools there.

kuhnel updated this revision to Diff 304087.Nov 10 2020, 1:28 AM

removed stray edit to Dockerfile

kbobyrev added inline comments.Nov 10 2020, 1:33 AM
buildbot/osuosl/master/config/workers.py
211

Also, is this the correct reference? terraform/main.tf seems to have Windows buildbot configs https://reviews.llvm.org/rZORG0e2cebb40db7ff0a5e10f02041f1c3a6c106b937

@kbobyrev Yes, the main.tf was a dirty working copy. I fixed that now.

This revision was not accepted when it landed; it landed in state Needs Review.Nov 11 2020, 12:40 AM
This revision was automatically updated to reflect the committed changes.