[PATCH] D144651: [Serialization] Place command line defines in the correct file

John Brawn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 5 10:56:02 PDT 2023


john.brawn updated this revision to Diff 511153.
john.brawn edited the summary of this revision.
john.brawn added a comment.

New version that checks for special filenames in ResolveImportedPath. I spent a while trying to come up with a test using just clang where not doing this fails, but couldn't as it looks like the failure is specific to objective-c module handing in lldb.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144651/new/

https://reviews.llvm.org/D144651

Files:
  clang-tools-extra/clangd/index/SymbolCollector.cpp
  clang/docs/ReleaseNotes.rst
  clang/lib/Serialization/ASTReader.cpp
  clang/lib/Serialization/ASTWriter.cpp
  clang/test/PCH/macro-cmdline.c
  clang/test/PCH/ms-pch-macro.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D144651.511153.patch
Type: text/x-patch
Size: 5472 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230405/a1f9dbc0/attachment.bin>


More information about the cfe-commits mailing list