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

Yaxun Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 28 08:53:29 PDT 2023


yaxunl accepted this revision.
yaxunl added a comment.
This revision is now accepted and ready to land.

LGTM. Thanks.



================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUCtorDtorLowering.cpp:76
+    if (!AlreadyRegistered)
+      IRB.CreateCall(ConstructorTy, CS->getOperand(1));
   }
----------------
I noticed these functions are called not following the priority.

However, I guess that is out of the scope of this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149340



More information about the llvm-commits mailing list