[PATCH] D40900: [Sanitizers, test] Fix sanitizer tests on Solaris (PR 33274)

Kamil Rytarowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 17 04:41:19 PST 2018


krytarowski added a comment.

@ro For your interest, just in case that it's in the scope of your interest:
there is a work towards merging your work with Illumos and ship in pkgsrc.

http://mail-index.netbsd.org/tech-pkg/2018/01/16/msg019322.html

There are also patches generated locally for SmartOS... it would be great to merge them
instead of keeping in downstream branches.



================
Comment at: test/asan/TestCases/alloca_constant_size.cc:14
 # define alloca _alloca
-#elif defined(__FreeBSD__) || defined(__NetBSD__)
-#include <stdlib.h>
-#else
+
+#if defined(__sun__) && defined(__svr4__)
----------------
I've added `#endif` here as a hotfix.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D40900





More information about the llvm-commits mailing list