This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix up make bootstrap-lean; make install (PR bootstrap/58572)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Paolo Bonzini <bonzini at gnu dot org>
- Cc: DJ Delorie <dj at redhat dot com>, Alexandre Oliva <aoliva at redhat dot com>, Ralf Wildenhues <Ralf dot Wildenhues at gmx dot de>, gcc-patches at gcc dot gnu dot org, Tom Tromey <tromey at redhat dot com>
- Date: Fri, 7 Mar 2014 11:45:52 +0100
- Subject: Re: [PATCH] Fix up make bootstrap-lean; make install (PR bootstrap/58572)
- Authentication-results: sourceware.org; auth=none
- References: <20140306185849 dot GF22862 at tucnak dot redhat dot com> <531974B7 dot 7070300 at gnu dot org>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Fri, Mar 07, 2014 at 08:26:47AM +0100, Paolo Bonzini wrote:
> Il 06/03/2014 19:58, Jakub Jelinek ha scritto:
> >Bootstrapped/regtested on x86_64-linux and i686-linux and tested also
> >with make bootstrap-lean; make install, ok for trunk?
>
> Nice fix! Did you check that on a regular bootstrap (formerly
> bubblestrap) you can modify the source code for the affected headers
> in libstdc++-v3, and alias.o/dfp.o will be rebuilt?
It isn't rebuilt with the patch, but I don't see how it is any different
from all the other -isystem headers (e.g. the gcc/include headers like
stddef.h etc. that we use -isystem for too).
I'm not sure it is worth bothering, while there are occassional changes
to those headers, it is at most a few occurrences a year and usually just
addition of new stuff, after all the headers should remain ABI compatible.
Anyway, bootstrapping/regtesting now this variant where it only changes
things for *-lean cases and not for normal bootstrap.
2014-03-07 Jakub Jelinek <jakub@redhat.com>
PR bootstrap/58572
* Makefile.tpl (POSTSTAGE1_CXX_EXPORT): Use -isystem instead of
-I for libstdc++-v3 includes if $(LEAN).
* Makefile.in: Regenerated.
--- Makefile.tpl.jj 2013-11-11 22:38:29.000000000 +0100
+++ Makefile.tpl 2014-03-07 10:56:42.221105929 +0100
@@ -242,9 +242,9 @@ POSTSTAGE1_CXX_EXPORT = \
-B$$r/$(HOST_SUBDIR)/prev-gcc/ -B$(build_tooldir)/bin/ -nostdinc++ \
-B$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/src/.libs \
-B$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/libsupc++/.libs \
- -I$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/include/$(TARGET_SUBDIR) \
- -I$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/include \
- -I$$s/libstdc++-v3/libsupc++ \
+ `if $(LEAN); then echo ' -isystem '; else echo ' -I'; fi`$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/include/$(TARGET_SUBDIR) \
+ `if $(LEAN); then echo ' -isystem '; else echo ' -I'; fi`$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/include \
+ `if $(LEAN); then echo ' -isystem '; else echo ' -I'; fi`$$s/libstdc++-v3/libsupc++ \
-L$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/src/.libs \
-L$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/libsupc++/.libs"; \
export CXX; \
--- Makefile.in.jj 2013-11-11 22:38:21.000000000 +0100
+++ Makefile.in 2014-03-07 10:56:13.059107667 +0100
@@ -239,9 +239,9 @@ POSTSTAGE1_CXX_EXPORT = \
-B$$r/$(HOST_SUBDIR)/prev-gcc/ -B$(build_tooldir)/bin/ -nostdinc++ \
-B$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/src/.libs \
-B$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/libsupc++/.libs \
- -I$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/include/$(TARGET_SUBDIR) \
- -I$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/include \
- -I$$s/libstdc++-v3/libsupc++ \
+ `if $(LEAN); then echo ' -isystem '; else echo ' -I'; fi`$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/include/$(TARGET_SUBDIR) \
+ `if $(LEAN); then echo ' -isystem '; else echo ' -I'; fi`$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/include \
+ `if $(LEAN); then echo ' -isystem '; else echo ' -I'; fi`$$s/libstdc++-v3/libsupc++ \
-L$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/src/.libs \
-L$$r/prev-$(TARGET_SUBDIR)/libstdc++-v3/libsupc++/.libs"; \
export CXX; \
Jakub