[llvm] fa74df3 - Revert [llvm] annotate ABIBreakingChecks symbols for DLL export (#147965)

via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 10 08:21:05 PDT 2025


Author: Andrew Rogers
Date: 2025-07-10T08:21:02-07:00
New Revision: fa74df38ade4053534731ea1e00ffe900e9e9492

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

LOG: Revert [llvm] annotate ABIBreakingChecks symbols for DLL export (#147965)

This patch reverts commit 24475409e4eac6fd60e2111424a4bef3452c8f21. The
change introduced a cyclic dependency:
```
fatal error: cyclic dependency in module 'LLVM_Utils': LLVM_Utils -> LLVM_Config_ABI_Breaking -> LLVM_Utils
```

See failure log at
https://green.lab.llvm.org/job/llvm.org/job/clang-stage2-Rthinlto/976/console.

Added: 
    

Modified: 
    llvm/include/llvm/Config/abi-breaking.h.cmake

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/Config/abi-breaking.h.cmake b/llvm/include/llvm/Config/abi-breaking.h.cmake
index 318bd015f80d2..2d27e02b1d545 100644
--- a/llvm/include/llvm/Config/abi-breaking.h.cmake
+++ b/llvm/include/llvm/Config/abi-breaking.h.cmake
@@ -12,9 +12,6 @@
 #ifndef LLVM_ABI_BREAKING_CHECKS_H
 #define LLVM_ABI_BREAKING_CHECKS_H
 
-// Compiler.h is required for LLVM_ABI definition.
-#include "llvm/Support/Compiler.h"
-
 /* Define to enable checks that alter the LLVM C++ ABI */
 #cmakedefine01 LLVM_ENABLE_ABI_BREAKING_CHECKS
 
@@ -46,12 +43,12 @@
 #endif
 namespace llvm {
 #if LLVM_ENABLE_ABI_BREAKING_CHECKS
-LLVM_ABI extern int EnableABIBreakingChecks;
+extern int EnableABIBreakingChecks;
 LLVM_HIDDEN_VISIBILITY
 __attribute__((weak)) int *VerifyEnableABIBreakingChecks =
     &EnableABIBreakingChecks;
 #else
-LLVM_ABI extern int DisableABIBreakingChecks;
+extern int DisableABIBreakingChecks;
 LLVM_HIDDEN_VISIBILITY
 __attribute__((weak)) int *VerifyDisableABIBreakingChecks =
     &DisableABIBreakingChecks;


        


More information about the llvm-commits mailing list