[llvm] [asan] Disable instrumentation for available_externally global with COFF (PR #81109)

via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 8 01:14:15 PST 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff b662c9aa0e0580e1fc78a8787414c86ad1984742 d403f55d10561a430bd72b1ca63bac24df166dce -- llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp b/llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp
index 80643e5b04..46f7a8f71e 100644
--- a/llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp
+++ b/llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp
@@ -1957,7 +1957,7 @@ bool ModuleAddressSanitizer::shouldInstrumentGlobal(GlobalVariable *G) const {
     // On COFF, don't instrument non-ODR linkages.
     if (G->isInterposable())
       return false;
-    // If the global has AvailableExternally linkage, then it is not in this 
+    // If the global has AvailableExternally linkage, then it is not in this
     // module, which means it does not need to be instrumented.
     if (G->hasAvailableExternallyLinkage())
       return false;

``````````

</details>


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


More information about the llvm-commits mailing list