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

https://github.com/kadircet closed https://github.com/llvm/llvm-project/pull/72246 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

https://github.com/kadircet updated https://github.com/llvm/llvm-project/pull/72246 From 84f320a1c746eddea3171bd984613464a5872cbb Mon Sep 17 00:00:00 2001 From: Kadir Cetinkaya Date: Tue, 14 Nov 2023 13:08:52 +0100 Subject: [PATCH] [include-cleaner] Make sure exports of stdlib also works for p

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

https://github.com/kadircet edited https://github.com/llvm/llvm-project/pull/72246 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

@@ -232,6 +232,17 @@ class PragmaIncludes::RecordPragma : public PPCallbacks, public CommentHandler { void checkForExport(FileID IncludingFile, int HashLine, std::optional IncludedHeader, OptionalFileEntryRef IncludedFile) { +a

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

@@ -232,6 +232,17 @@ class PragmaIncludes::RecordPragma : public PPCallbacks, public CommentHandler { void checkForExport(FileID IncludingFile, int HashLine, std::optional IncludedHeader, OptionalFileEntryRef IncludedFile) { +a

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

https://github.com/hokein approved this pull request. https://github.com/llvm/llvm-project/pull/72246 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

@@ -232,6 +232,17 @@ class PragmaIncludes::RecordPragma : public PPCallbacks, public CommentHandler { void checkForExport(FileID IncludingFile, int HashLine, std::optional IncludedHeader, OptionalFileEntryRef IncludedFile) { +a

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

@@ -232,6 +232,17 @@ class PragmaIncludes::RecordPragma : public PPCallbacks, public CommentHandler { void checkForExport(FileID IncludingFile, int HashLine, std::optional IncludedHeader, OptionalFileEntryRef IncludedFile) { +a

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

https://github.com/hokein edited https://github.com/llvm/llvm-project/pull/72246 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

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

https://github.com/kadircet created https://github.com/llvm/llvm-project/pull/72246 This was creating discrepancy in cases where we might have a physical file entry (e.g. because we followed a source location from a stdlib file) and tried to find its exporters. From 49ea3ae3468c5d67b2b6b03339d