This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Enable relative configs in check mode
ClosedPublic

Authored by kadircet on Jul 29 2021, 10:41 PM.

Details

Summary

See https://github.com/clangd/clangd/issues/649#issuecomment-885903316.
Also disables config support in lit tests to make sure tests are not affected by
clangd config files lying around.

Diff Detail

Event Timeline

kadircet created this revision.Jul 29 2021, 10:41 PM
kadircet requested review of this revision.Jul 29 2021, 10:41 PM
sammccall accepted this revision.Jul 30 2021, 5:08 AM
sammccall added inline comments.
clang-tools-extra/clangd/tool/Check.cpp
275–278

Only if FakeFile.empty() - we shouldn't take the arbitrary temp path into account

This revision is now accepted and ready to land.Jul 30 2021, 5:08 AM
kadircet updated this revision to Diff 363040.Jul 30 2021, 5:23 AM
kadircet marked an inline comment as done.
  • Don't use local configs for temp files
This revision was landed with ongoing or failed builds.Jul 30 2021, 5:29 AM
This revision was automatically updated to reflect the committed changes.