[llvm] [HWASan] [NFC] Avoid unnecessary vector (PR #119898)
Florian Mayer via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 17 10:21:57 PST 2024
https://github.com/fmayer updated https://github.com/llvm/llvm-project/pull/119898
>From e5018cdcefbc52ff6bb60b996bfada6185e2258e Mon Sep 17 00:00:00 2001
From: Florian Mayer <fmayer at google.com>
Date: Fri, 13 Dec 2024 08:33:26 -0800
Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia?=
=?UTF-8?q?l=20version?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Created using spr 1.3.4
---
.../Instrumentation/HWAddressSanitizer.cpp | 25 ++++++++-----------
1 file changed, 10 insertions(+), 15 deletions(-)
diff --git a/llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp b/llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp
index 530061e3b6bb7d..6333b1b6ba86f4 100644
--- a/llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp
+++ b/llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp
@@ -1793,8 +1793,15 @@ void HWAddressSanitizer::instrumentGlobal(GlobalVariable *GV, uint8_t Tag) {
}
void HWAddressSanitizer::instrumentGlobals() {
- std::vector<GlobalVariable *> Globals;
- for (GlobalVariable &GV : M.globals()) {
+ MD5 Hasher;
+ Hasher.update(M.getSourceFileName());
+ MD5::MD5Result Hash;
+ Hasher.final(Hash);
+ uint8_t Tag = Hash[0];
+
+ assert(TagMaskByte >= 16);
+
+ for (GlobalVariable &GV : make_early_inc_range(M.globals())) {
if (GV.hasSanitizerMetadata() && GV.getSanitizerMetadata().NoHWAddress)
continue;
@@ -1812,24 +1819,12 @@ void HWAddressSanitizer::instrumentGlobals() {
if (GV.hasSection())
continue;
- Globals.push_back(&GV);
- }
-
- MD5 Hasher;
- Hasher.update(M.getSourceFileName());
- MD5::MD5Result Hash;
- Hasher.final(Hash);
- uint8_t Tag = Hash[0];
-
- assert(TagMaskByte >= 16);
-
- for (GlobalVariable *GV : Globals) {
// Don't allow globals to be tagged with something that looks like a
// short-granule tag, otherwise we lose inter-granule overflow detection, as
// the fast path shadow-vs-address check succeeds.
if (Tag < 16 || Tag > TagMaskByte)
Tag = 16;
- instrumentGlobal(GV, Tag++);
+ instrumentGlobal(&GV, Tag++);
}
}
More information about the llvm-commits
mailing list