[PATCH] D149340: [AMDGPU] Place global constructors in .init_array and .fini_array

[Yaxun Liu via Phabricator via llvm-commits]

yaxunl added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUCtorDtorLowering.cpp:76
+    if (!AlreadyRegistered)
+      IRB.CreateCall(ConstructorTy, CS->getOperand(1));
   }
----------------
jhuber6 wrote:
> yaxunl wrote:
> > I noticed these functions are called not following the priority.
> > 
> > However, I guess that is out of the scope of this patch.
> True, it was like that when I got here. Do you know who the current user is for this feature?
Currently, all HIP programs use this feature when -fsanitize=addr is used.

But they do not care about priority yet.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149340