commit ed2a613f363578aae3ac1414faa25b46fde2fda9
author Vitaly Buka <vitalybuka@google.com> Thu May 20 16:32:09 2021 -0700
committer Copybara-Service <copybara-worker@google.com> Tue May 25 19:11:12 2021 -0700
tree 2bdd049ea85e6da3389f5723b46d3515cb8ab4e0
parent d902290b550673dddb179f8a6697018831f1dfb5

[scudo] Try to re-enabled the test on arm

It's probably fixed by D102886.

Builder to watch https://lab.llvm.org/buildbot/#/builders/clang-cmake-armv7-full

Reviewed By: hctim, cryptoad

Differential Revision: https://reviews.llvm.org/D102887

GitOrigin-RevId: 1c6272a481fda287f9b506a83ed21d74131742af

tests/secondary_test.cpp

diff --git a/tests/secondary_test.cpp b/tests/secondary_test.cpp
index 608dfd8..c50101a 100644
--- a/tests/secondary_test.cpp
+++ b/tests/secondary_test.cpp
@@ -33,11 +33,6 @@
   EXPECT_GE(SecondaryT::getBlockSize(P), Size);
   L->deallocate(scudo::Options{}, P);
 
-// TODO(hctim): Looks like the secondary pointer doesn't get unmapped on arm32.
-// It's not clear whether this is a kernel issue, or something in EXPECT_DEATH()
-// is mmap-ing something that uses the same vaddr space. For now, just disable
-// the test on arm32 until we can debug it further.
-#ifndef __arm__
   // If the Secondary can't cache that pointer, it will be unmapped.
   if (!L->canCache(Size)) {
     EXPECT_DEATH(
@@ -52,7 +47,6 @@
         },
         "");
   }
-#endif // __arm__
 
   const scudo::uptr Align = 1U << 16;
   P = L->allocate(scudo::Options{}, Size + Align, Align);