diff options
author | Queen Dela Cruz <qdelacru@cisco.com> | 2021-09-16 10:17:37 +0200 |
---|---|---|
committer | Kadir Cetinkaya <kadircet@google.com> | 2021-09-16 10:27:15 +0200 |
commit | 9c4a1686d7c487fff8e63fa67e64623eea8986d5 (patch) | |
tree | 0c008360893a1483b3391cd9b2431d31eae534fd /lldb/source/Commands/CommandObjectPlatform.cpp | |
parent | 6a5f49a1acf861357b9ccb8495e4416878a2a7ad (diff) | |
download | llvm-9c4a1686d7c487fff8e63fa67e64623eea8986d5.zip llvm-9c4a1686d7c487fff8e63fa67e64623eea8986d5.tar.gz llvm-9c4a1686d7c487fff8e63fa67e64623eea8986d5.tar.bz2 |
[clangd] Fix clangd crash when including a header
Fixes https://github.com/clangd/clangd/issues/819
SourceLocation of macros change when a header file is included above it. This is not checked when creating a PreamblePatch, resulting in reusing previously built preamble with an incorrect source location for the macro in the example test case.
This patch stores the SourceLocation in the struct TextualPPDirective so that it gets checked when comparing old vs new preambles.
Also creates a preamble patch for code completion parsing so that clangd does not crash when following the example test case with a large file.
Reviewed By: kadircet
Differential Revision: https://reviews.llvm.org/D108045
Diffstat (limited to 'lldb/source/Commands/CommandObjectPlatform.cpp')
0 files changed, 0 insertions, 0 deletions