[Openmp-commits] [PATCH] D77609: [OpenMP] Added the support for hidden helper task in RTL

Wed Mar 17 07:57:21 PDT 2021

lebedev.ri added a comment.

In D77609#2631767 <https://reviews.llvm.org/D77609#2631767>, @protze.joachim wrote:

> In D77609#2629372 <https://reviews.llvm.org/D77609#2629372>, @lebedev.ri wrote:
>
>> I believe this involves compiling a program that uses omp with clang, linking it to llvm's libomp,
>> and linking it to some library that is compiled with gcc and linked to libgomp.
>> The issue appears to happen regardless of whether or not the libgomp used is provided at runtime by gcc or llvm.
>
> Linking two OpenMP runtime libraries into one application is guaranteed to break things. You have basically no way to guarantee that calls to API functions go to the right runtime.

Presumably you have read my comment in it's entirety, and did see that both libgomp and libomp used are from llvm?
What's the point of llvm's libgomp then?

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77609/new/

https://reviews.llvm.org/D77609