[clang] [TySan] Don't report globals with external storage. (PR #120565)

via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 19 04:13:17 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 cffe22a93726a64e6a205b5dcd1c306a62488412 a12283a7da8da16053f02ecf4f41c2eaa06717cd --extensions cpp -- clang/test/CodeGen/sanitize-type-globals.cpp clang/lib/CodeGen/SanitizerMetadata.cpp
``````````

</details>

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

``````````diff
diff --git a/clang/lib/CodeGen/SanitizerMetadata.cpp b/clang/lib/CodeGen/SanitizerMetadata.cpp
index a2da7ff8ce..61fdf3399f 100644
--- a/clang/lib/CodeGen/SanitizerMetadata.cpp
+++ b/clang/lib/CodeGen/SanitizerMetadata.cpp
@@ -107,8 +107,7 @@ void SanitizerMetadata::reportGlobal(llvm::GlobalVariable *GV,
 
   GV->setSanitizerMetadata(Meta);
 
-  if (Ty.isNull() ||
-      !CGM.getLangOpts().Sanitize.has(SanitizerKind::Type) ||
+  if (Ty.isNull() || !CGM.getLangOpts().Sanitize.has(SanitizerKind::Type) ||
       NoSanitizeAttrMask & SanitizerKind::Type)
     return;
 

``````````

</details>


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


More information about the cfe-commits mailing list