[PATCH] D64943: [Clang][OpenMP offload] Eliminate use of OpenMP linker script

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 16 07:59:09 PDT 2019


ABataev added a comment.

In D64943#1633170 <https://reviews.llvm.org/D64943#1633170>, @lebedev.ri wrote:

> In D64943#1633164 <https://reviews.llvm.org/D64943#1633164>, @ABataev wrote:
>
> > In D64943#1619958 <https://reviews.llvm.org/D64943#1619958>, @sdmitriev wrote:
> >
> > > As I understand ‘atexit’ solution would be target dependent (‘__cxa_atexit’ on Linux and ‘atexit’ on Windows) whereas @llvm.global_ctors/dtors variables offer similar and platform neutral functionality (http://llvm.org/docs/LangRef.html#the-llvm-global-ctors-global-variable). Why do you think that ‘atexit’ is a better choice?
> >
> >
> > Because it does not work on Windows, better to have portable solution, if possible.
>
>
> Is there a bug # ?


@vzakhari?


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

https://reviews.llvm.org/D64943





More information about the cfe-commits mailing list