[llvm] Simplify hot-path size computations in BumpPtrAllocator. (PR #101467)

Owen Anderson via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 1 03:03:56 PDT 2024


https://github.com/resistor updated https://github.com/llvm/llvm-project/pull/101467

>From 5ff2016e1197760edffb47162d7e070897c93dfb Mon Sep 17 00:00:00 2001
From: Owen Anderson <resistor at mac.com>
Date: Tue, 30 Jul 2024 21:50:15 -1000
Subject: [PATCH] Simplify hot-path size computations in BumpPtrAllocator.

~0.1% instruction count improvements

https://llvm-compile-time-tracker.com/compare.php?from=07d2709a17860a202d91781769a88837e4fb5f2a&to=d5cc47831ecd9f0a2b164b16da67f74b94e9aafc&stat=instructions:u
---
 llvm/include/llvm/Support/Allocator.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/llvm/include/llvm/Support/Allocator.h b/llvm/include/llvm/Support/Allocator.h
index 6be7c320a26f8..568f0d34032fa 100644
--- a/llvm/include/llvm/Support/Allocator.h
+++ b/llvm/include/llvm/Support/Allocator.h
@@ -169,9 +169,9 @@ class BumpPtrAllocatorImpl
       // Update the allocation point of this memory block in MemorySanitizer.
       // Without this, MemorySanitizer messages for values originated from here
       // will point to the allocation of the entire slab.
-      __msan_allocated_memory(AlignedPtr, Size);
+      __msan_allocated_memory(reinterpret_cast<char *>(AlignedPtr), Size);
       // Similarly, tell ASan about this space.
-      __asan_unpoison_memory_region(AlignedPtr, Size);
+      __asan_unpoison_memory_region(reinterpret_cast<char *>(AlignedPtr), Size);
       return reinterpret_cast<char *>(AlignedPtr);
     }
 



More information about the llvm-commits mailing list