[compiler-rt] [scudo] Added test fixture for cache tests. (PR #102230)

Christopher Ferris via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 6 15:03:30 PDT 2024


================
@@ -265,3 +265,99 @@ TEST_F(MapAllocatorWithReleaseTest, SecondaryThreadsRace) {
   Allocator->getStats(&Str);
   Str.output();
 }
+
+struct MapAllocatorCacheTest : public Test {
+  static constexpr scudo::u32 MarkerBytes = 0xDEADBEEF;
+
+  static void testUnmapCallback(scudo::MemMapT &MemMap) {
+    scudo::u32 *Ptr = reinterpret_cast<scudo::u32 *>(MemMap.getBase());
+    *Ptr = MarkerBytes;
+  }
+
+  using Config = scudo::DefaultConfig;
----------------
cferris1000 wrote:

I would use the value directly then.

However, is there a reason you are using the DefaultConfig? I am always worried about using something that might change and cause weird failurs.

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


More information about the llvm-commits mailing list