public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Mark Mitchell <mark@codesourcery.com>
To: gcc-patches@gcc.gnu.org
Subject: Re: RFA: Improve 128-bit long double configure test
Date: Mon, 26 Nov 2007 21:09:00 -0000	[thread overview]
Message-ID: <474B0A8F.2010308@codesourcery.com> (raw)
In-Reply-To: <20071126005021.GA28587@caradoc.them.org>

Daniel Jacobowitz wrote:
> On Fri, Nov 23, 2007 at 05:52:43PM -0800, Mark Mitchell wrote:
>> Assuming that we don't want to move away from probing here, I have a
>> couple of questions about your patch.  It looks like we've got
>> target-probing code for __stack_chk_fail right aboe the code you changed
>> that does the same kind of search for GLIBC headers.  I'd expect that
>> your change should be applied to that code too, or, better, that the
>> code that finds GLIBC headers should be factored out.
> 
> It was - that's why there were two pieces of the patch.

Sorry, I must have missed that.  I apologize.

>> Also, why wouldn't your method (using CPP) going to work on all
>> host/target combinations?  If so, we should use it everywhere,
>> rather than just in the ($host = $target) != $build case.
> 
> It relies on the compiler being GCC; I don't know if I can get away
> with that for any other compiler one might use to build a
> Linux-targeted GCC.

That makes sense.  Personally, I'd be happy to stop trying to support
building GCC with non-GCC compilers, but I understand that's certain to
be controversial and I'm not advocating it.

I would like to hear from the Power maintainers, though, about whether
we can just remove this attempt at auto-probing the target altogether,
though.  I'd rather make people building the compiler work pass an
explicit configuration option and simplify this logic.

Thanks,

-- 
Mark Mitchell
CodeSourcery
mark@codesourcery.com
(650) 331-3385 x713

  reply	other threads:[~2007-11-26 18:04 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-15 20:18 Daniel Jacobowitz
2007-11-24 10:06 ` Mark Mitchell
2007-11-26  6:45   ` Daniel Jacobowitz
2007-11-26 21:09     ` Mark Mitchell [this message]
2008-03-06 20:33 ` [ping] " Daniel Jacobowitz
2008-04-09  0:40   ` Mark Mitchell
2008-04-09  2:59     ` David Edelsohn
2008-04-09  6:48       ` Mark Mitchell
2008-04-09 13:16         ` David Edelsohn
2008-04-09 13:37           ` Paolo Bonzini
2008-04-09 16:02           ` Mark Mitchell

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=474B0A8F.2010308@codesourcery.com \
    --to=mark@codesourcery.com \
    --cc=gcc-patches@gcc.gnu.org \
    /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).