<div dir="ltr">do you know why this was not caught during manual testing before commit? </div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 10, 2016 at 9:04 PM, Alexey Samsonov via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: samsonov<br>
Date: Thu Mar 10 23:04:49 2016<br>
New Revision: 263195<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=263195&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=263195&view=rev</a><br>
Log:<br>
Fix ASan test cases after r263177<br>
<br>
Modified:<br>
    compiler-rt/trunk/test/asan/TestCases/printf-2.c<br>
    compiler-rt/trunk/test/asan/TestCases/strcasestr-2.c<br>
    compiler-rt/trunk/test/asan/TestCases/strstr-2.c<br>
<br>
Modified: compiler-rt/trunk/test/asan/TestCases/printf-2.c<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/printf-2.c?rev=263195&r1=263194&r2=263195&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/printf-2.c?rev=263195&r1=263194&r2=263195&view=diff</a><br>
==============================================================================<br>
--- compiler-rt/trunk/test/asan/TestCases/printf-2.c (original)<br>
+++ compiler-rt/trunk/test/asan/TestCases/printf-2.c Thu Mar 10 23:04:49 2016<br>
@@ -1,9 +1,9 @@<br>
 // RUN: %clang_asan -O2 %s -o %t<br>
-// We need replace_str=0 and replace_intrin=0 to avoid reporting errors in<br>
-// strlen() and memcpy() called by printf().<br>
-// RUN: %env_asan_opts=replace_str=0:replace_intrin=0:check_printf=1 not %run %t 2>&1 | FileCheck --check-prefix=CHECK-ON %s<br>
-// RUN: %env_asan_opts=replace_str=0:replace_intrin=0:check_printf=0 %run %t 2>&1 | FileCheck --check-prefix=CHECK-OFF %s<br>
-// RUN: %env_asan_opts=replace_str=0:replace_intrin=0 not %run %t 2>&1 | FileCheck --check-prefix=CHECK-ON %s<br>
+// We need replace_str=0, intercept_strlen=0 and replace_intrin=0 to avoid<br>
+// reporting errors in strlen() and memcpy() called by printf().<br>
+// RUN: %env_asan_opts=replace_str=0:intercept_strlen=0:replace_intrin=0:check_printf=1 not %run %t 2>&1 | FileCheck --check-prefix=CHECK-ON %s<br>
+// RUN: %env_asan_opts=replace_str=0:intercept_strlen=0:replace_intrin=0:check_printf=0 %run %t 2>&1 | FileCheck --check-prefix=CHECK-OFF %s<br>
+// RUN: %env_asan_opts=replace_str=0:intercept_strlen=0:replace_intrin=0 not %run %t 2>&1 | FileCheck --check-prefix=CHECK-ON %s<br>
<br>
 // FIXME: printf is not intercepted on Windows yet.<br>
 // XFAIL: win32<br>
<br>
Modified: compiler-rt/trunk/test/asan/TestCases/strcasestr-2.c<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/strcasestr-2.c?rev=263195&r1=263194&r2=263195&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/strcasestr-2.c?rev=263195&r1=263194&r2=263195&view=diff</a><br>
==============================================================================<br>
--- compiler-rt/trunk/test/asan/TestCases/strcasestr-2.c (original)<br>
+++ compiler-rt/trunk/test/asan/TestCases/strcasestr-2.c Thu Mar 10 23:04:49 2016<br>
@@ -3,7 +3,7 @@<br>
<br>
 // Test intercept_strstr asan option<br>
 // Disable other interceptors because strlen may be called inside strcasestr<br>
-// RUN: %env_asan_opts=intercept_strstr=false:replace_str=false %run %t 2>&1<br>
+// RUN: %env_asan_opts=intercept_strstr=false:replace_str=false:intercept_strlen=false %run %t 2>&1<br>
<br>
 // There's no interceptor for strcasestr on Windows<br>
 // XFAIL: win32<br>
<br>
Modified: compiler-rt/trunk/test/asan/TestCases/strstr-2.c<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/strstr-2.c?rev=263195&r1=263194&r2=263195&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/strstr-2.c?rev=263195&r1=263194&r2=263195&view=diff</a><br>
==============================================================================<br>
--- compiler-rt/trunk/test/asan/TestCases/strstr-2.c (original)<br>
+++ compiler-rt/trunk/test/asan/TestCases/strstr-2.c Thu Mar 10 23:04:49 2016<br>
@@ -3,7 +3,7 @@<br>
<br>
 // Test intercept_strstr asan option<br>
 // Disable other interceptors because strlen may be called inside strstr<br>
-// RUN: %env_asan_opts=intercept_strstr=false:replace_str=false %run %t 2>&1<br>
+// RUN: %env_asan_opts=intercept_strstr=false:replace_str=false:intercept_strlen=false %run %t 2>&1<br>
<br>
 #include <assert.h><br>
 #include <string.h><br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>