This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PATCH: Fix the relax finalize pass


On Tue, May 27, 2003 at 07:59:56AM -0700, H. J. Lu wrote:
> On Tue, May 27, 2003 at 10:31:31AM -0400, Daniel Jacobowitz wrote:
> > On Thu, May 15, 2003 at 11:36:16AM -0400, Daniel Jacobowitz wrote:
> > > On Thu, May 15, 2003 at 07:53:34AM -0700, H. J. Lu wrote:
> > > > On Thu, May 15, 2003 at 03:43:42PM +0930, Alan Modra wrote:
> > > > > On Wed, May 14, 2003 at 10:41:08PM -0700, H. J. Lu wrote:
> > > > > > On Wed, May 14, 2003 at 10:31:36PM -0700, Richard Henderson wrote:
> > > > > > > On Wed, May 14, 2003 at 07:19:12PM -0700, H. J. Lu wrote:
> > > > > > > > It sounds good. Are you going to implement it?
> > > > > > > 
> > > > > > > Not any time soon.
> > > > > > > 
> > > > > > 
> > > > > > How about this kludge in the meantime?
> > > > > 
> > > > > No, just disable DATA_SEGMENT_ALIGN for ia64 until you can get the
> > > > > ia64 backend fixed.  Not defining COMMONPAGESIZE should do the trick.
> > > > > 
> > > > 
> > > > I will check in this patch now.
> > > > 
> > > > H.J.
> > > 
> > > Does anyone know if this is necessary on the 2.14 branch also?
> > 
> > Anyone?  Is there a testcase for this problem in the testsuite?
> > 
> 
> I kept a complete gcc build directory (854M) just to be able to
> reproduce the problem :-(.

For lack of a certain answer, I think the safe thing to do is to merge
the patch below to the branch.  Does anypone disagree?

> 
> 
> > > > 2003-05-15  H.J. Lu <hongjiu.lu@intel.com>
> > > > 
> > > > 	* emulparams/elf64_ia64.sh: Don't set COMMONPAGESIZE for now.
> > > > 
> > > > --- ld/emulparams/elf64_ia64.sh.relax	2002-02-22 23:44:00.000000000 -0800
> > > > +++ ld/emulparams/elf64_ia64.sh	2003-05-15 07:48:57.000000000 -0700
> > > > @@ -7,7 +7,9 @@ OUTPUT_FORMAT="elf64-ia64-little"
> > > >  ARCH=ia64
> > > >  MACHINE=
> > > >  MAXPAGESIZE=0x10000
> > > > -if test -n "$CREATE_SHLIB"; then
> > > > +# FIXME: It interferes with linker relaxation. Disable it until it is
> > > > +# fixed.
> > > > +if test "0" = "1" -a -n "$CREATE_SHLIB"; then
> > > >    # Optimize shared libraries for 16K page size
> > > >    COMMONPAGESIZE=0x4000
> > > >  fi
> > > 
> > > 
> > > -- 
> > > Daniel Jacobowitz
> > > MontaVista Software                         Debian GNU/Linux Developer
> > > 
> > 
> > -- 
> > Daniel Jacobowitz
> > MontaVista Software                         Debian GNU/Linux Developer
> 

-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]