Page MenuHomePhabricator
Differential D127844

[clangd] Pull suppression logic into common path, apply for driver diagnostics
Needs ReviewPublic
Actions

Authored by kadircet on Jun 15 2022, 4:19 AM.

Details

Reviewers
ilya-biryukov
sammccall
Summary

Fixes https://github.com/clangd/clangd/issues/1142

Diff Detail

Event Timeline

kadircet created this revision.Jun 15 2022, 4:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 15 2022, 4:19 AM
Herald added subscribers: usaxena95, arphaman. · View Herald Transcript
kadircet requested review of this revision.Jun 15 2022, 4:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 15 2022, 4:19 AM
Herald added subscribers: cfe-commits, MaskRay. · View Herald Transcript
kadircet updated this revision to Diff 437109.Jun 15 2022, 4:27 AM

Just pass optional directly

Harbormaster completed remote builds in B169950: Diff 437109.Jun 15 2022, 4:49 AM
nridge added a subscriber: nridge.Jun 20 2022, 12:05 AM
kadircet added a comment.Dec 6 2022, 1:07 AM

ping

kadircet added a reviewer: sammccall.Jan 10 2023, 6:27 AM

Revision Contents
Changeset List

PathSize
clang-tools-extra/
clangd/
4 lines
56 lines
83 lines
33 lines
unittests/
16 lines
16 lines
21 lines

Diff 437109

View Options

clang-tools-extra/clangd/Diagnostics.h

Loading...
View Options

clang-tools-extra/clangd/Diagnostics.cpp

Loading...
View Options

clang-tools-extra/clangd/ParsedAST.cpp

Loading...
View Options

clang-tools-extra/clangd/Preamble.cpp

Loading...
View Options

clang-tools-extra/clangd/unittests/CompilerTests.cpp

Loading...
View Options

clang-tools-extra/clangd/unittests/ConfigCompileTests.cpp

Loading...
View Options

clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp

Loading...