This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Remove the temporary alias for clangd::DiagnosticConsumer.
ClosedPublic

Authored by hokein on Jan 29 2020, 3:47 AM.

Diff Detail

Event Timeline

hokein created this revision.Jan 29 2020, 3:47 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 29 2020, 3:47 AM

We have updated our internal code to use the new name.

Unit tests: pass. 62283 tests passed, 0 failed and 827 were skipped.

clang-tidy: pass.

clang-format: pass.

Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml

Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.

hokein updated this revision to Diff 241122.Jan 29 2020, 5:16 AM

remove the deprecated contructor as well.

sammccall accepted this revision.Jan 29 2020, 5:23 AM

Thanks for the cleanup!

This revision is now accepted and ready to land.Jan 29 2020, 5:23 AM

Unit tests: pass. 62283 tests passed, 0 failed and 827 were skipped.

clang-tidy: pass.

clang-format: pass.

Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml

Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.

This revision was automatically updated to reflect the committed changes.