rjmccall added a comment. Thanks for the comment.
================ Comment at: lib/CodeGen/CGCUDANV.cpp:444 + auto HandleValue = + CtorBuilder.CreateAlignedLoad(GpuBinaryHandle, CGM.getPointerAlign()); + llvm::Constant *Zero = llvm::Constant::getNullValue(HandleValue->getType()); ---------------- yaxunl wrote: > rjmccall wrote: > > yaxunl wrote: > > > rjmccall wrote: > > > > Do you not need to worry about concurrency here? > > > The ctor functions are executed by the dynamic loader before the program > > > gains the control. The dynamic loader cannot excute the ctor functions > > > concurrently since doing that would not gurantee thread safety of the > > > loaded program. Therefore we can assume sequential execution of ctor > > > functions here. > > Okay. That's worth a comment. > > > > Is the name here specified by some ABI document, or is it just a > > conventional name that we're picking now? > Will add a comment for that. > > You mean `__hip_gpubin_handle`? It is an implementation detail. It is not > defined by ABI or other documentation. Since it is only used internally by > ctor functions, it is not a visible elf symbol. Its name is by convention > since the cuda corresponding one was named __cuda_gpubin_handle. Well, it *is* ABI, right? It's necessary for all translation units to agree to use the same symbol here or else the registration will happen multiple times. https://reviews.llvm.org/D49083 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
