[clang-tools-extra] [include-cleaner] Make sure exports of stdlib also works for physical files (PR #72246)

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 14 10:39:41 PST 2023


================
@@ -232,6 +232,17 @@ class PragmaIncludes::RecordPragma : public PPCallbacks, public CommentHandler {
   void checkForExport(FileID IncludingFile, int HashLine,
                       std::optional<Header> IncludedHeader,
                       OptionalFileEntryRef IncludedFile) {
+    auto AddExport = [&] {
----------------
hokein wrote:

Can you inline this lambda? I think the code is correct, but I found it a bit hard to reason about the change. 

https://github.com/llvm/llvm-project/pull/72246


More information about the cfe-commits mailing list