Commit 90e4ebdc authored by protze@itc.rwth-aachen.de's avatar protze@itc.rwth-aachen.de
Browse files

[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
parent 6e01cd67

openmp/runtime/test/ompt/synchronization/reduction/tree_reduce.c

+7 −3
Original line number Diff line number Diff line 
// 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: