This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Include the correct header for typeid()
ClosedPublic

Authored by nridge on Jan 8 2023, 12:18 AM.

Diff Detail

Event Timeline

nridge created this revision.Jan 8 2023, 12:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 8 2023, 12:18 AM
Herald added a subscriber: arphaman. · View Herald Transcript
nridge requested review of this revision.Jan 8 2023, 12:18 AM
philnik added a subscriber: philnik.Jan 8 2023, 1:38 AM
kadircet accepted this revision.Jan 9 2023, 1:05 AM

thanks for catching this!

clang-tools-extra/clangd/unittests/DiagnosticsTests.cpp
1531 ↗(On Diff #487148)

nit: i'd actually drop the test, as this is too much of a detail to test against.

This revision is now accepted and ready to land.Jan 9 2023, 1:05 AM
nridge updated this revision to Diff 487571.Jan 9 2023, 3:06 PM

Remove test

This revision was landed with ongoing or failed builds.Jan 9 2023, 3:07 PM
This revision was automatically updated to reflect the committed changes.