[llvm] [AMDGPU] Call the `FINI_ARRAY` destructors in the correct order (PR #71815)

Jon Chesterfield via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 9 12:07:15 PST 2023


JonChesterfield wrote:

This needs tests with multiple translation units. Multiple IR files with appending linkage would also be fine. It also needs tests that _don't_ force the ordering with priority.

Destructors need to fire in reverse order of constructors. If your comment is accurate to the implementation:

> The destructors stored in FINI_ARRAY are actually stored in reverse order

Then that won't happen for all of the above test cases.

https://github.com/llvm/llvm-project/pull/71815


More information about the llvm-commits mailing list