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

Mészáros Gergely via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Jan 18 09:01:05 PST 2023

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.



More information about the Openmp-commits mailing list