- ed0ba496fe01eb8edfa86beade8a37768e7c12ef Updates the API for Exception Safety testing to use build... by Abseil Team <absl-team@google.com>

  - c4b7a4e517c9404932c45f2f9f92eb7dc694e45d Internal change by Abseil Team <absl-team@google.com>
  - 76c78ed9385f65d881511645446e0bb8ababf6ec Add missing ABSL_PREDICT_FALSE to one of FixedArray::at()... by Abseil Team <absl-team@google.com>
  - 1204fb1c46f007dd9dfb7d9abf3e96c58835d193 Internal change. by Greg Falcon <gfalcon@google.com>
  - f1f47c98a026bc5e425ae83ff4a2eb391bbd3d9b Add internal-only functionality to examine the stack, to ... by Derek Mauro <dmauro@google.com>
  - 30d63097cd268d912f917526f6511005580465c4 fix typo by Abseil Team <absl-team@google.com>
  - 942d7efa6cf54cd248ca57dcaf3c245188b52a76 Remove unnecessary semicolons from comment examples. by Abseil Team <absl-team@google.com>
  - 7db0669cf23a06d934d3ed8c76aee4e4e23b7e04 Remove malloc_hook and malloc_extension from our internal... by Greg Falcon <gfalcon@google.com>
  - 0190f1063d101b1ded355019df2e1d325931f6c7 Make the maximum length of a string view equal difference... by Abseil Team <absl-team@google.com>
  - c8ae37cbce29449b02115a0ebd435ddc3d7ab062 Add namespace qualification. by Shaindel Schwartz <shaindel@google.com>
  - ff70afe2e6e3dd39f51ce9829e3e1f18231bf4d7 Fix internal/direct_mmap.h for non-linux builds. by Greg Falcon <gfalcon@google.com>

GitOrigin-RevId: ed0ba496fe01eb8edfa86beade8a37768e7c12ef
Change-Id: I7595ee3480d1d6724fd3797c15ba9d9be0d17e62

1 files changed, 2 insertions, 50 deletions

diff --git a/absl/base/internal/low_level_alloc_test.cc b/absl/base/internal/low_level_alloc_test.cc
index 7c359f306115..cf2b363299f4 100644
--- a/absl/base/internal/low_level_alloc_test.cc
+++ b/absl/base/internal/low_level_alloc_test.cc
@@ -21,8 +21,6 @@
 #include <unordered_map>
 #include <utility>
 
-#include "absl/base/internal/malloc_hook.h"
-
 namespace absl {
 namespace base_internal {
 namespace {
@@ -139,58 +137,12 @@ static void Test(bool use_new_arena, bool call_malloc_hook, int n) {
     TEST_ASSERT(LowLevelAlloc::DeleteArena(arena));
   }
 }
-
-// used for counting allocates and frees
-static int32_t allocates;
-static int32_t frees;
-
-// ignore uses of the allocator not triggered by our test
-static std::thread::id* test_tid;
-
-// called on each alloc if kCallMallocHook specified
-static void AllocHook(const void *p, size_t size) {
-  if (using_low_level_alloc) {
-    if (*test_tid == std::this_thread::get_id()) {
-      allocates++;
-    }
-  }
-}
-
-// called on each free if kCallMallocHook specified
-static void FreeHook(const void *p) {
-  if (using_low_level_alloc) {
-    if (*test_tid == std::this_thread::get_id()) {
-      frees++;
-    }
-  }
-}
-
 // LowLevelAlloc is designed to be safe to call before main().
 static struct BeforeMain {
   BeforeMain() {
-    test_tid = new std::thread::id(std::this_thread::get_id());
-    TEST_ASSERT(MallocHook::AddNewHook(&AllocHook));
-    TEST_ASSERT(MallocHook::AddDeleteHook(&FreeHook));
-    TEST_ASSERT(allocates == 0);
-    TEST_ASSERT(frees == 0);
     Test(false, false, 50000);
-    TEST_ASSERT(allocates != 0);  // default arena calls hooks
-    TEST_ASSERT(frees != 0);
-    for (int i = 0; i != 16; i++) {
-      bool call_hooks = ((i & 1) == 1);
-      allocates = 0;
-      frees = 0;
-      Test(true, call_hooks, 15000);
-      if (call_hooks) {
-        TEST_ASSERT(allocates > 5000);  // arena calls hooks
-        TEST_ASSERT(frees > 5000);
-      } else {
-        TEST_ASSERT(allocates == 0);  // arena doesn't call hooks
-        TEST_ASSERT(frees == 0);
-      }
-    }
-    TEST_ASSERT(MallocHook::RemoveNewHook(&AllocHook));
-    TEST_ASSERT(MallocHook::RemoveDeleteHook(&FreeHook));
+    Test(true, false, 50000);
+    Test(true, true, 50000);
   }
 } before_main;