[PATCH] D91622: [sanitizer_common][test] Disable CombinedAllocator32Compact etc. on Solaris/sparcv9

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 17 14:14:05 PST 2020


vitalybuka added a comment.

Have you tried to change kAddressSpaceSize above?



================
Comment at: compiler-rt/lib/sanitizer_common/tests/sanitizer_allocator_test.cpp:942
 
+#if !(SANITIZER_SOLARIS && defined(__sparcv9))
 TEST(SanitizerCommon, SizeClassAllocator32Iteration) {
----------------
Better approach is to use:
```
#if defined(_MSC_VER) && defined(_DLL)
#define MAYBE_StrDupOOBTest DISABLED_StrDupOOBTest
#else
#define MAYBE_StrDupOOBTest StrDupOOBTest
#endif

TEST(AddressSanitizer, MAYBE_StrDupOOBTest) {
```
this way the test at least compiles on the platform.

But it's going to make the file inconsistent.
So you like you can update the file with MAYBE_ in a separate patch.
But feel free to keep as-is.






Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91622/new/

https://reviews.llvm.org/D91622



More information about the llvm-commits mailing list