[PATCH] [Msan] Fix the DynRet unit test to build on FreeBSD

Viktor Kutuzov vkutuzov at accesssoftek.com
Wed Jan 21 03:30:26 PST 2015


Hi kcc, samsonov, emaste, eugenis,

There's no clearenv() on FreeBSD.

http://reviews.llvm.org/D7086

Files:
  lib/msan/tests/msan_test.cc

Index: lib/msan/tests/msan_test.cc
===================================================================
--- lib/msan/tests/msan_test.cc
+++ lib/msan/tests/msan_test.cc
@@ -511,10 +511,9 @@
 
 TEST(MemorySanitizer, DynRet) {
   ReturnPoisoned<S8>();
-  EXPECT_NOT_POISONED(clearenv());
+  EXPECT_NOT_POISONED(atoi("0"));
 }
 
-
 TEST(MemorySanitizer, DynRet1) {
   ReturnPoisoned<S8>();
 }

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D7086.18493.patch
Type: text/x-patch
Size: 388 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150121/97b94811/attachment.bin>


More information about the llvm-commits mailing list