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
next prev parent 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).