[compiler-rt] r275539 - [asan] Reduce flakiness of halt_on_error-torture.cc testcase.
Maxim Ostapenko via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 26 01:45:12 PDT 2016
On 22/07/16 04:13, Kostya Serebryany via llvm-commits wrote:
> Do you think this is fully fixed?
> I've seen it fail on my local machine just now (passes when I repeat)
Oh, sorry, I've missed this mail (that nasty corporate mail changing).
I'll take a look on this.
>
> On Fri, Jul 15, 2016 at 12:57 AM, Maxim Ostapenko via llvm-commits
> <llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>> wrote:
>
> Author: chefmax
> Date: Fri Jul 15 02:57:43 2016
> New Revision: 275539
>
> URL: http://llvm.org/viewvc/llvm-project?rev=275539&view=rev
> Log:
> [asan] Reduce flakiness of halt_on_error-torture.cc testcase.
>
> It seems in some situations we have clashes on very first error so
> test only prints "nested bug in the same thread, aborting" rather
> than "use-after-poison", so remove corresponding " RUN: FileCheck
> %s < 10.txt" line.
> Also, the two last " RUN: FileCheck --check-prefix=CHECK-COLLISION
> %s < 1.txt || FileCheck --check-prefix=CHECK-NO-COLLISION %s <
> 1.txt" look wrong, they should check 10.txt. Fix these lines too.
>
> Differential Revision: https://reviews.llvm.org/D22309
>
> Modified:
> compiler-rt/trunk/test/asan/TestCases/Posix/halt_on_error-torture.cc
>
> Modified:
> compiler-rt/trunk/test/asan/TestCases/Posix/halt_on_error-torture.cc
> URL:
> http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/Posix/halt_on_error-torture.cc?rev=275539&r1=275538&r2=275539&view=diff
> ==============================================================================
> ---
> compiler-rt/trunk/test/asan/TestCases/Posix/halt_on_error-torture.cc
> (original)
> +++
> compiler-rt/trunk/test/asan/TestCases/Posix/halt_on_error-torture.cc
> Fri Jul 15 02:57:43 2016
> @@ -9,15 +9,11 @@
> //
> // Collisions are unlikely but still possible so we need the ||.
> // RUN:
> %env_asan_opts=halt_on_error=false:suppress_equal_pcs=false %run
> %t 10 20 >10.txt 2>&1 || true
> -// This one is racy although _very_ unlikely to fail:
> -// RUN: FileCheck %s < 10.txt
> -// RUN: FileCheck --check-prefix=CHECK-COLLISION %s < 1.txt ||
> FileCheck --check-prefix=CHECK-NO-COLLISION %s < 1.txt
> +// RUN: FileCheck --check-prefix=CHECK-COLLISION %s < 10.txt ||
> FileCheck --check-prefix=CHECK-NO-COLLISION %s < 10.txt
> //
> // Collisions are unlikely but still possible so we need the ||.
> // RUN: %env_asan_opts=halt_on_error=false %run %t 10 20 >10.txt
> 2>&1 || true
> -// This one is racy although _very_ unlikely to fail:
> -// RUN: FileCheck %s < 10.txt
> -// RUN: FileCheck --check-prefix=CHECK-COLLISION %s < 1.txt ||
> FileCheck --check-prefix=CHECK-NO-COLLISION %s < 1.txt
> +// RUN: FileCheck --check-prefix=CHECK-COLLISION %s < 10.txt ||
> FileCheck --check-prefix=CHECK-NO-COLLISION %s < 10.txt
>
> #include <stdio.h>
> #include <stdlib.h>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
More information about the llvm-commits
mailing list