[clang] [clang][ExtractAPI] Remove erroneous module name check in MacroCallbacks (PR #107059)

via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 3 01:52:35 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Daniel Grumberg (daniel-grumberg)

<details>
<summary>Changes</summary>

rdar://135044923

---
Full diff: https://github.com/llvm/llvm-project/pull/107059.diff


1 Files Affected:

- (modified) clang/lib/ExtractAPI/ExtractAPIConsumer.cpp (+1-1) 


``````````diff
diff --git a/clang/lib/ExtractAPI/ExtractAPIConsumer.cpp b/clang/lib/ExtractAPI/ExtractAPIConsumer.cpp
index 0adc23280fd6c0..75c2dec22400b9 100644
--- a/clang/lib/ExtractAPI/ExtractAPIConsumer.cpp
+++ b/clang/lib/ExtractAPI/ExtractAPIConsumer.cpp
@@ -350,7 +350,7 @@ class APIMacroCallback : public MacroCallback {
   bool shouldMacroBeIncluded(const SourceLocation &MacroLoc,
                              StringRef ModuleName) override {
     // Do not include macros from external files
-    return LCF(MacroLoc) || API.ProductName == ModuleName;
+    return LCF(MacroLoc);
   }
 
 private:

``````````

</details>


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


More information about the cfe-commits mailing list