[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
This commit is contained in:
Vitaly Buka 2021-05-20 16:32:09 -07:00
parent d0bc04d6b9
commit 1c6272a481
1 changed files with 0 additions and 6 deletions

View File

@ -33,11 +33,6 @@ template <typename Config> static void testSecondaryBasic(void) {
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 @@ template <typename Config> static void testSecondaryBasic(void) {
},
"");
}
#endif // __arm__
const scudo::uptr Align = 1U << 16;
P = L->allocate(scudo::Options{}, Size + Align, Align);