[Openmp-commits] [PATCH] D142008: [libomptarget] Delay recursive shared library registrations until offload RTLs are loaded

[Mészáros Gergely via Phabricator via Openmp-commits]

Maetveis added a comment.

In D142008#4062603 <https://reviews.llvm.org/D142008#4062603>, @jdoerfert wrote:

> This is too complex and we should not need static thread local stuff at all.
> Why don't we set the flag that we loaded libomptarget early. If we end up loading it again we should just not initialize it. All we really want is to reduce the scope of the lock guarding the init flag, no?

Yes but the problem is that we didn't finish initializing it yet when the second call comes in, we're still inside the initialization of the amdgpu plugin so that one and any others after it didn't finish initialization yet.


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

https://reviews.llvm.org/D142008