public inbox for java-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
To: Paolo Bonzini <bonzini@gnu.org>
Cc: "Joseph S. Myers" <joseph@codesourcery.com>,
	gcc-patches@gcc.gnu.org,
	       Ralf Wildenhues <Ralf.Wildenhues@gmx.de>,
	       Ian Lance Taylor <iant@google.com>,
	Steve Ellcey <sje@cup.hp.com>,
	       Richard Earnshaw <richard.earnshaw@arm.com>,
	       Ramana Radhakrishnan <ramana.radhakrishnan@arm.com>,
	       Nick Clifton <nickc@redhat.com>,
	Douglas Rupp <rupp@gnat.com>,
	       Tristan Gingold <gingold@adacore.com>,
	       Mike Stump <mikestump@comcast.net>,
	Kaz Kojima <kkojima@gcc.gnu.org>,
	       David Edelsohn <dje.gcc@gmail.com>,
	       Sterling Augustine <augustine.sterling@gmail.com>,
	       Arnaud Charlet <charlet@adacore.com>,
	java-patches@gcc.gnu.org,
	       Nicola Pero <nicola.pero@meta-innovation.com>,
	libstdc++@gcc.gnu.org,
	       Richard Sandiford <rdsandiford@googlemail.com>
Subject: Re: [build] Move unwinder to toplevel libgcc (v2)
Date: Thu, 11 Aug 2011 15:26:00 -0000	[thread overview]
Message-ID: <yddhb5oj7l3.fsf@manam.CeBiTec.Uni-Bielefeld.DE> (raw)
In-Reply-To: <4E42B494.9010309@gnu.org> (Paolo Bonzini's message of "Wed, 10	Aug 2011 18:40:52 +0200")

Paolo Bonzini <bonzini@gnu.org> writes:

> On 08/10/2011 06:05 PM, Rainer Orth wrote:
>>>> >>  True: it is called once per multilib.
>>> >
>>> >  Just to doublecheck, are we sure that unwind.h is always the same?
>> Yep: it's unwind-generic.h for almost all targets, just a few arm
>> targets use config/arm/unwind-arm.h for all multilibs.
>
> Patch doing rm -f is preapproved then.

Here's what I installed, after making libgcc/unwind-generic.h read-only,
reconfiguring and rebuilding first without and with the patch and
observe the bug gone on i386-pc-solaris2.10.

	Rainer


2011-08-11  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>

	* Makefile.in (install-unwind_h): Remove destination file first.

diff --git a/libgcc/Makefile.in b/libgcc/Makefile.in
--- a/libgcc/Makefile.in
+++ b/libgcc/Makefile.in
@@ -994,6 +994,7 @@ gcc-extra-parts:
 all: $(extra-parts)
 
 install-unwind_h:
+	rm -f $(gcc_objdir)/include/unwind.h
 	cp unwind.h $(gcc_objdir)/include/unwind.h
 	chmod a+r $(gcc_objdir)/include/unwind.h
 
-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University

  reply	other threads:[~2011-08-11 15:23 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-03 13:33 Rainer Orth
2011-08-03 16:07 ` Nick Clifton
2011-08-03 21:11 ` Benjamin Kosnik
2011-08-05  9:43 ` Paolo Bonzini
2011-08-05  9:48   ` Andrew Haley
2011-08-05 11:46   ` Rainer Orth
2011-08-05 12:52     ` Andrew Haley
2011-08-05 13:33     ` Paolo Bonzini
     [not found] ` <201108052018.38537.mikael.morin@sfr.fr>
2011-08-05 19:49   ` Paolo Bonzini
     [not found]     ` <201108061243.19056.mikael.morin@sfr.fr>
2011-08-06 14:32       ` Paolo Bonzini
2011-08-10 13:47 ` Joseph S. Myers
2011-08-10 13:51   ` Rainer Orth
2011-08-10 13:57     ` Joseph S. Myers
2011-08-10 15:11       ` Rainer Orth
2011-08-10 16:05         ` Paolo Bonzini
2011-08-10 16:10           ` Rainer Orth
2011-08-10 17:00             ` Paolo Bonzini
2011-08-11 15:26               ` Rainer Orth [this message]
     [not found]                 ` <CAMe9rOrznkTrvxg-KC4TrNF5uSua654p_1R0GZ-9rkNUF5Z76A@mail.gmail.com>
2011-08-12  7:28                   ` Paolo Bonzini
     [not found]             ` <201108101750.43530.pedro@codesourcery.com>
2011-08-11 12:06               ` Paolo Bonzini
2011-08-11 14:25                 ` Rainer Orth
2011-08-11 14:38                   ` Joseph S. Myers
2011-08-11 14:42                     ` Paolo Bonzini
2011-08-11 15:23                       ` Joseph S. Myers
2011-08-11 15:32                         ` Rainer Orth
2011-08-11 16:18                           ` Paolo Bonzini
2011-08-11 19:53                             ` Rainer Orth
2011-08-11 14:44                     ` Rainer Orth

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=yddhb5oj7l3.fsf@manam.CeBiTec.Uni-Bielefeld.DE \
    --to=ro@cebitec.uni-bielefeld.de \
    --cc=Ralf.Wildenhues@gmx.de \
    --cc=augustine.sterling@gmail.com \
    --cc=bonzini@gnu.org \
    --cc=charlet@adacore.com \
    --cc=dje.gcc@gmail.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=gingold@adacore.com \
    --cc=iant@google.com \
    --cc=java-patches@gcc.gnu.org \
    --cc=joseph@codesourcery.com \
    --cc=kkojima@gcc.gnu.org \
    --cc=libstdc++@gcc.gnu.org \
    --cc=mikestump@comcast.net \
    --cc=nickc@redhat.com \
    --cc=nicola.pero@meta-innovation.com \
    --cc=ramana.radhakrishnan@arm.com \
    --cc=rdsandiford@googlemail.com \
    --cc=richard.earnshaw@arm.com \
    --cc=rupp@gnat.com \
    --cc=sje@cup.hp.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).