[Openmp-commits] [PATCH] D25510: Fix for mistake done by https://reviews.llvm.org/D23115

Jonas Hahnfeld via Openmp-commits openmp-commits at lists.llvm.org
Mon Oct 17 00:16:19 PDT 2016


Hahnfeld accepted this revision.
Hahnfeld added a comment.
This revision is now accepted and ready to land.

LGTM, I think this was left over from my previous tries to fix the problem.

When exactly does this happen? Something like this?

  #pragma omp parallel num_threads(2)
  #pragma omp single
  {
    #pragma omp task if(0)
    { }
  }

Would it be possible to add a test case for this? Maybe with a destructor of a `firstprivate` C++ object?


Repository:
  rL LLVM

https://reviews.llvm.org/D25510





More information about the Openmp-commits mailing list