[compiler-rt] r280823 - [powerpc] Disable ManyThreadsWithStatsStressTest test for ppc64

Mike Aizatsky via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 7 14:32:25 PDT 2016


Bill,

This breaks the build by creating unused variables:

http://lab.llvm.org:8011/builders/sanitizer-ppc64be-linux/builds/3703


/home/buildbots/ppc64be-sanitizer/sanitizer-ppc64be/build/llvm/projects/compiler-rt/lib/asan/tests/asan_interface_test.cc:105:21:
error: unused variable 'kManyThreadsNumThreads'
[-Werror,-Wunused-const-variable]

static const size_t kManyThreadsNumThreads =
                    ^
/home/buildbots/ppc64be-sanitizer/sanitizer-ppc64be/build/llvm/projects/compiler-rt/lib/asan/tests/asan_interface_test.cc:108:14:
error: unused function 'ManyThreadsWithStatsWorker'
[-Werror,-Wunused-function]
static void *ManyThreadsWithStatsWorker(void *arg) {
             ^
2 errors generated.
l




On Wed, Sep 7, 2016 at 9:38 AM Bill Seurer via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Author: seurer
> Date: Wed Sep  7 11:30:31 2016
> New Revision: 280823
>
> URL: http://llvm.org/viewvc/llvm-project?rev=280823&view=rev
> Log:
> [powerpc] Disable ManyThreadsWithStatsStressTest test for ppc64
>
> Since r279664 this test causes frequent failures of test runs for ppc64le
> and
> occasional failures for ppc64be which makes buildbot results unreliable.
> If
> the underlying problem is fixed it can be re-enabled.
>
> Modified:
>     compiler-rt/trunk/lib/asan/tests/asan_interface_test.cc
>
> Modified: compiler-rt/trunk/lib/asan/tests/asan_interface_test.cc
> URL:
> http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/asan/tests/asan_interface_test.cc?rev=280823&r1=280822&r2=280823&view=diff
>
> ==============================================================================
> --- compiler-rt/trunk/lib/asan/tests/asan_interface_test.cc (original)
> +++ compiler-rt/trunk/lib/asan/tests/asan_interface_test.cc Wed Sep  7
> 11:30:31 2016
> @@ -117,6 +117,9 @@ static void *ManyThreadsWithStatsWorker(
>    return 0;
>  }
>
> +#ifndef __powerpc64__
> +// FIXME: This has not reliably worked on powerpc since r279664.
> Re-enable
> +// this once the problem is tracked down and fixed.
>  TEST(AddressSanitizerInterface, ManyThreadsWithStatsStressTest) {
>    size_t before_test, after_test, i;
>    pthread_t threads[kManyThreadsNumThreads];
> @@ -133,6 +136,7 @@ TEST(AddressSanitizerInterface, ManyThre
>    // so we can't check for equality here.
>    EXPECT_LT(after_test, before_test + (1UL<<20));
>  }
> +#endif
>
>  static void DoDoubleFree() {
>    int *x = Ident(new int);
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-- 
Mike
Sent from phone
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160907/6998bbc9/attachment.html>


More information about the llvm-commits mailing list