[PATCH] D99291: [AIX] Support init priority attribute

Jason Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 7 15:55:10 PDT 2021


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

LGTM.



================
Comment at: clang/lib/CodeGen/CGDeclCXX.cpp:663
 
+  // Create our global prioritized cleanup function.
+  if (!PrioritizedCXXStermFinalizers.empty()) {
----------------
Just noting that this trunk of code have very similar logic counter part in `EmitCXXGlobalInitFunc`. It seems there is no easy way to common them up given the current way of implementing it. It may require a non-trivial amount of refactoring for us to common up the similar logic between init and cleanup. 


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

https://reviews.llvm.org/D99291



More information about the cfe-commits mailing list