Bug 31546 - add --enable-intermodule
Summary: add --enable-intermodule
Status: RESOLVED FIXED
Alias: None
Product: gcc
Classification: Unclassified
Component: libfortran (show other bugs)
Version: 4.3.0
: P3 enhancement
Target Milestone: 4.3.0
Assignee: Bernhard Reutner-Fischer
URL:
Keywords:
Depends on: 31537
Blocks:
  Show dependency treegraph
 
Reported: 2007-04-12 15:02 UTC by Bernhard Reutner-Fischer
Modified: 2007-09-21 19:50 UTC (History)
3 users (show)

See Also:
Host:
Target:
Build:
Known to work:
Known to fail:
Last reconfirmed: 2007-04-17 20:07:38


Attachments
patch to add --enable-intermodule to libgfortran (953 bytes, patch)
2007-04-12 15:03 UTC, Bernhard Reutner-Fischer
Details | Diff
updated patch (1.43 KB, patch)
2007-04-15 12:38 UTC, Bernhard Reutner-Fischer
Details | Diff
add enable-intermodule for libgfortran against r127717 (1.27 KB, patch)
2007-08-22 20:53 UTC, Bernhard Reutner-Fischer
Details | Diff
add missing includes of config.h to avoid conflicts in prototypes for LFS funcs (554 bytes, patch)
2007-08-25 22:33 UTC, Bernhard Reutner-Fischer
Details | Diff
add --enable-intermodule for libgfortran (809 bytes, patch)
2007-09-21 15:54 UTC, Bernhard Reutner-Fischer
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Bernhard Reutner-Fischer 2007-04-12 15:02:41 UTC
add option to build libgfortran and fortran with -combine.

This creates a smaller binary and may also create a faster binary. The former is the main motivation from my POV.
Comment 1 Bernhard Reutner-Fischer 2007-04-12 15:03:41 UTC
Created attachment 13356 [details]
patch to add --enable-intermodule to libgfortran
Comment 2 Francois-Xavier Coudert 2007-04-13 15:12:31 UTC
(In reply to comment #0)
> This creates a smaller binary and may also create a faster binary. The former
> is the main motivation from my POV.

Do you have figures to justify these two claims?
Comment 3 Bernhard Reutner-Fischer 2007-04-15 12:38:53 UTC
Created attachment 13367 [details]
updated patch

Updated patch.
Comment 4 Bernhard Reutner-Fischer 2007-04-15 12:39:57 UTC
(In reply to comment #2)
> (In reply to comment #0)
> > This creates a smaller binary and may also create a faster binary. The former
> > is the main motivation from my POV.
> 
> Do you have figures to justify these two claims?
> 

   text	   data	    bss	    dec	    hex	filename
 601912	   2636	    284	 604832	  93aa0	../gcc-4.3.orig/./i686-linux-gnu/libgfortran/.libs/libgfortran.so.3.0.0
 562768	   2348	    288	 565404	  8a09c	./i686-linux-gnu/libgfortran/.libs/libgfortran.so.3.0.0

I do not claim that it is faster since i didn't measure this aspect.
Comment 5 Bernhard Reutner-Fischer 2007-08-22 20:53:00 UTC
Created attachment 14096 [details]
add enable-intermodule for libgfortran against r127717

Updated patch to allow for configuring libgfortran with ./configure --enable-intermodule optimization
Comment 6 Bernhard Reutner-Fischer 2007-08-25 22:33:55 UTC
Created attachment 14116 [details]
add missing includes of config.h to avoid conflicts in prototypes for LFS funcs

gfortran-4.3-PR31546-config.h-inc.01.diff

2007-08-26  Bernhard Fischer  <me>

      PR fortran/31546
      * (configure.ac): Add --enable-intermodule for onestep build.
      * (Makefile.am): Handle onestep build.
      * (libgfortran.h, intrinsics/abort.c, intrinsics/iso_c_binding.c,
      intrinsics/string_intrinsics.c): Include config.h before system headers.
Comment 7 info@herberouge.be 2007-08-25 22:35:36 UTC
Subject: Re:  add --enable-intermodule

STOP your email please !
----- Original Message ----- 
From: "aldot at gcc dot gnu dot org" <gcc-bugzilla@gcc.gnu.org>
To: <gcc-bugs@gcc.gnu.org>
Sent: Sunday, August 26, 2007 12:33 AM
Subject: [Bug libfortran/31546] add --enable-intermodule


>
>
> ------- Comment #6 from aldot at gcc dot gnu dot org  2007-08-25 
> 22:33 -------
> Created an attachment (id=14116)
> --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14116&action=view)
> add missing includes of config.h to avoid conflicts in prototypes for LFS 
> funcs
>
> gfortran-4.3-PR31546-config.h-inc.01.diff
>
> 2007-08-26  Bernhard Fischer  <me>
>
>      PR fortran/31546
>      * (configure.ac): Add --enable-intermodule for onestep build.
>      * (Makefile.am): Handle onestep build.
>      * (libgfortran.h, intrinsics/abort.c, intrinsics/iso_c_binding.c,
>      intrinsics/string_intrinsics.c): Include config.h before system 
> headers.
>
>
> -- 
>
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31546
>
>
> 
Comment 8 Francois-Xavier Coudert 2007-09-14 11:37:08 UTC
Wasn't the patch for this approved? (http://gcc.gnu.org/ml/fortran/2007-08/msg00517.html) Since stage3 has started, it probably would be best to commit rather soon.
Comment 9 Bernhard Reutner-Fischer 2007-09-21 15:54:17 UTC
Created attachment 14238 [details]
add --enable-intermodule for libgfortran

2007-09-21  Bernhard Fischer  <aldot>

      PR fortran/31546
      * (configure.ac): Add --enable-intermodule for onestep build.
      * (Makefile.am): Handle onestep build.
      * (configure, Makefile.in): Regenerate.
Comment 10 Bernhard Reutner-Fischer 2007-09-21 15:56:26 UTC
fx, I cannot apply this patch since i cannot autoreconf with the appropriate versions of autoconf and automake. Can you apply it with the regenerated files, please (and assign this bug to you, if you want)?

TIA,
Comment 11 Francois-Xavier Coudert 2007-09-21 19:49:46 UTC
Subject: Bug 31546

Author: fxcoudert
Date: Fri Sep 21 19:49:34 2007
New Revision: 128654

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=128654
Log:
	PR fortran/31546
	* (configure.ac): Add --enable-intermodule for onestep build.
	* (Makefile.am): Handle onestep build.
	* (configure, Makefile.in): Regenerate.

Modified:
    trunk/libgfortran/ChangeLog
    trunk/libgfortran/Makefile.am
    trunk/libgfortran/Makefile.in
    trunk/libgfortran/configure
    trunk/libgfortran/configure.ac

Comment 12 Francois-Xavier Coudert 2007-09-21 19:50:59 UTC
Fixed.