emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: /srv/bzr/emacs/emacs-24 r111251: * pre-crt0.c (data_start): Initiali


From: Paul Eggert
Subject: Re: /srv/bzr/emacs/emacs-24 r111251: * pre-crt0.c (data_start): Initialize to 1.
Date: Tue, 12 Feb 2013 14:06:59 -0800
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2

On 02/12/13 13:07, Glenn Morris wrote:
> the code is used by other, working platforms.

Can you please give an example of such a platform?
I tried to find an example but came up dry (please see below).

> Why is this change safe/necessary for emacs-24 at this late stage?

The change is clearly safe as the variable's contents are
never used.  That being said, the change is not needed on a
platform that defines BSD_SYSTEM or DATA_START or
ORDINARY_LINK or CANNOT_DUMP or SYSTEM_MALLOC, as these
systems never use the address of the variable.  I just now
checked configure.ac etc. and it appears that the change is
therefore not needed on aix, *bsd, cygwin, darwin, gnu,
hpux*, irix6-5, ms-dos, ms-w32, sol2*, unixware.  Is this
the complete set of systems that Emacs ports to?  If so, the
change isn't needed and I'd like to revert it (though the
variable should be removed in the trunk, as it's no longer
used anywhere).  If not, the change is needed if the
variable is compiled with a newer GCC (or similar compiler)
with enough optimizing options that Emacs puts the variable
into bss.

I originally made the change because aix4-2 defined
ORDINARY_LINK but later AIX releases did not, which meant it
appeared that the variable was used on post-4.2 AIX
releases.  However, with my change to unexaix.c today, the
AIX port never uses the address of the variable, so the
change currently is not needed on post-4.2 AIX releases
after all.  I didn't put 2 and 2 together to see this;
sorry.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]