Created
June 16, 2011 18:17
-
-
Save ice799/1029857 to your computer and use it in GitHub Desktop.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
On Thu, Jun 16, 2011 at 09:15, H.J. Lu <[email protected]> wrote: | |
> On another platform, bad atexit implementation called from DSO caused | |
> weird run-time problems. The same problem can happen with glibc. The | |
> worse thing can happen with my patch is people will old compilers will | |
> get a link-time error, instead of strange run-time problems. | |
No. If there is no __dso_handle symbol the reference is NULL and | |
therefore atexit can work nicely when used in programs. It doesn't | |
work in DSOs but that was always the case before the introduction of | |
__dso_handle. | |
You didn't say why you wanted this change to happen. Just because of | |
this minute code improvement? That's not really a reason to | |
potentially break some situations. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment