public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Janis Johnson <janis187@us.ibm.com>
To: "H.J. Lu" <hjl@lucon.org>
Cc: gcc-patches@gcc.gnu.org, echristo@apple.com
Subject: Re: PATCH: PR testsuite/32932: ssp tests can't find libssp
Date: Tue, 31 Jul 2007 21:31:00 -0000	[thread overview]
Message-ID: <1185915163.6581.21.camel@janis-laptop> (raw)
In-Reply-To: <20070731182001.GA28810@lucon.org>

On Tue, 2007-07-31 at 11:20 -0700, H.J. Lu wrote:
> On Tue, Jul 31, 2007 at 10:43:56AM -0700, Janis Johnson wrote:
> > On Mon, 2007-07-30 at 16:03 -0700, H.J. Lu wrote:
> > > This patch makes sure that the newly built libssp is used for
> > > test. Tested on Linux/x86-64.
> > 
> > The patch doesn't handle multilibs; apparently you didn't test
> > with both -m32 and -m64.
> 
> It does support multilibs:
> 
> @@ -113,6 +114,10 @@ proc gcc_init { args } {
>      set gcc_error_prefix "error:"
> 
>      gcc_maybe_build_wrapper "${tmpdir}/gcc-testglue.o"
> +
> +    libssp_init [get_multilibs]

But libssp_init only processes the first one; see how they're
handled for libmudflap, for example, to add all of them to
LD_LIBRARY_PATH.

> and I tested with -m32 on Linux/x86-64:
> 
> Executing on host: /export/build/gnu/gcc/build-x86_64-linux/gcc/xgcc
>   -B/export/build/gnu/gcc/build-x86_64-linux/gcc/
>   /export/gnu/src/gcc/gcc/gcc/testsuite/gcc.dg/ssp-1.c -fstack-protector -fno-show-column
>   -L/export/build/gnu/gcc/build-x86_64-linux/x86_64-unknown-linux-gnu/32/libssp/.libs
>   -lm   -m32 -o ./ssp-1.exe    (timeout = 300)

This uses the 32-bit library for -m32, and adds it to LD_LIBRARY_PATH

> Executing on host: /export/build/gnu/gcc/build-x86_64-linux/gcc/xgcc
>   -B/export/build/gnu/gcc/build-x86_64-linux/gcc/
>   /export/gnu/src/gcc/gcc/gcc/testsuite/gcc.dg/ssp-1.c
>   -fstack-protector -fno-show-column
>   -L/export/build/gnu/gcc/build-x86_64-linux/x86_64-unknown-linux-gnu/32/libssp/.libs
>   -lm   -o ./ssp-1.exe    (timeout = 300)

This uses the 32-bit library for the default and adds the 32-bit
library to LD_LIBRARY_PATH.

Janis

  reply	other threads:[~2007-07-31 20:52 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-31  0:46 H.J. Lu
2007-07-31 17:53 ` Janis Johnson
2007-07-31 18:40   ` H.J. Lu
2007-07-31 21:31     ` Janis Johnson [this message]
2007-07-31 18:45   ` PATCH: gcc/testsuite: Set gcc_initialized to 1 H.J. Lu

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=1185915163.6581.21.camel@janis-laptop \
    --to=janis187@us.ibm.com \
    --cc=echristo@apple.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=hjl@lucon.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).