From 5288d7af5bc1b6775e122df5346f7cde9f65881d Mon Sep 17 00:00:00 2001 From: "protze@itc.rwth-aachen.de" Date: Tue, 4 Feb 2020 12:16:03 +0100 Subject: [OpenMP][OMPT] fix reduction test for 32-bit x86 Fixes [[ https://bugs.llvm.org/show_bug.cgi?id=44733 | TEST 'libomp :: ompt/synchronization/reduction/tree_reduce.c' FAILED on 32-bit x86 ]] For 32-bit we need at least 3 variables to avoid atomic reduction to be choosen by runtime function `__kmp_determine_reduction_method`. This patch adds reduction variables to the testcase. Reviewers: mgorny, Hahnfeld Differential Revision: https://reviews.llvm.org/D73850 (cherry picked from commit 90e4ebdce55fd3c1f8877f19784a5339b9890f98) --- .../runtime/test/ompt/synchronization/reduction/tree_reduce.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/openmp/runtime/test/ompt/synchronization/reduction/tree_reduce.c b/openmp/runtime/test/ompt/synchronization/reduction/tree_reduce.c index 2c73fe13900..847abc109b2 100644 --- a/openmp/runtime/test/ompt/synchronization/reduction/tree_reduce.c +++ b/openmp/runtime/test/ompt/synchronization/reduction/tree_reduce.c @@ -1,4 +1,5 @@ // RUN: %libomp-compile-and-run | %sort-threads | FileCheck %s +// RUN: %libomp-compile -DNOWAIT && %libomp-run | %sort-threads | FileCheck %s // REQUIRES: ompt // UNSUPPORTED: gcc #include "callback.h" @@ -11,14 +12,17 @@ #endif int main() { - int sum = 0; + int sum = 0, a = 0, b = 0; int i; #pragma omp parallel num_threads(5) -#pragma omp for reduction(+ : sum) FOR_CLAUSE +// for 32-bit architecture we need at least 3 variables to trigger tree +#pragma omp for reduction(+ : sum, a, b) FOR_CLAUSE for (i = 0; i < 10000; i++) { - sum += i; + a = b = sum += i; } + + printf("%i\n", sum); // CHECK: 0: NULL_POINTER=[[NULL:.*$]] // CHECK: {{^}}[[MASTER_ID:[0-9]+]]: ompt_event_parallel_begin: -- cgit v1.2.1