[clang] 3ddd186 - [Tooling] NFC, use const HeaderSearch for isSelfContainedHeader.

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 1 06:17:32 PDT 2023


Author: Haojian Wu
Date: 2023-06-01T15:17:21+02:00
New Revision: 3ddd18640af186bd46cb36064cf3bbe00b4b5a52

URL: https://github.com/llvm/llvm-project/commit/3ddd18640af186bd46cb36064cf3bbe00b4b5a52
DIFF: https://github.com/llvm/llvm-project/commit/3ddd18640af186bd46cb36064cf3bbe00b4b5a52.diff

LOG: [Tooling] NFC, use const HeaderSearch for isSelfContainedHeader.

Added: 
    

Modified: 
    clang/include/clang/Tooling/Inclusions/HeaderAnalysis.h
    clang/lib/Tooling/Inclusions/HeaderAnalysis.cpp

Removed: 
    


################################################################################
diff  --git a/clang/include/clang/Tooling/Inclusions/HeaderAnalysis.h b/clang/include/clang/Tooling/Inclusions/HeaderAnalysis.h
index 760b8dd0879c9..84d90c44de070 100644
--- a/clang/include/clang/Tooling/Inclusions/HeaderAnalysis.h
+++ b/clang/include/clang/Tooling/Inclusions/HeaderAnalysis.h
@@ -28,7 +28,7 @@ namespace tooling {
 /// This function can be expensive as it may scan the source code to find out
 /// dont-include-me pattern heuristically.
 bool isSelfContainedHeader(const FileEntry *FE, const SourceManager &SM,
-                           HeaderSearch &HeaderInfo);
+                           const HeaderSearch &HeaderInfo);
 
 /// This scans the given source code to see if it contains #import(s).
 bool codeContainsImports(llvm::StringRef Code);

diff  --git a/clang/lib/Tooling/Inclusions/HeaderAnalysis.cpp b/clang/lib/Tooling/Inclusions/HeaderAnalysis.cpp
index 49d23908d33b7..f83e19f10cbab 100644
--- a/clang/lib/Tooling/Inclusions/HeaderAnalysis.cpp
+++ b/clang/lib/Tooling/Inclusions/HeaderAnalysis.cpp
@@ -67,7 +67,7 @@ llvm::StringRef getFileContents(const FileEntry *FE, const SourceManager &SM) {
 } // namespace
 
 bool isSelfContainedHeader(const FileEntry *FE, const SourceManager &SM,
-                           HeaderSearch &HeaderInfo) {
+                           const HeaderSearch &HeaderInfo) {
   assert(FE);
   if (!HeaderInfo.isFileMultipleIncludeGuarded(FE) &&
       !HeaderInfo.hasFileBeenImported(FE) &&


        


More information about the cfe-commits mailing list