public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: "Joseph S. Myers" <joseph@codesourcery.com>
To: David Malcolm <dmalcolm@redhat.com>
Cc: Jeff Law <law@redhat.com>, <jit@gcc.gnu.org>, <gcc-patches@gcc.gnu.org>
Subject: Re: [PATCH 2/5] gcc: configure and Makefile changes needed by jit
Date: Wed, 15 Oct 2014 19:00:00 -0000	[thread overview]
Message-ID: <Pine.LNX.4.64.1410151842270.11645@digraph.polyomino.org.uk> (raw)
In-Reply-To: <1413397540.9513.125.camel@surprise>

On Wed, 15 Oct 2014, David Malcolm wrote:

> As for the "bindir" in site.exp, Joseph asked me when the library
> invokes a driver to convert from .s to .so to:
> 
> On Tue, 2014-09-23 at 23:27 +0000, Joseph S. Myers wrote:
> > * use the $(target_noncanonical)-gcc-$(version) name for the
> > driver rather than plain "gcc", to maximise the chance that it
> > is actually the same compiler the JIT library was built for (I
> > realise you may not actually depend on it being the same
> > compiler, but that does seem best; in principle in future it
> > should be possible to load multiple copies of the JIT library
> > to JIT for different targets, so that code for an offload
> > accelerator can go through the JIT).
> ( https://gcc.gnu.org/ml/jit/2014-q3/msg00033.html )
> 
> This full name is used when *installing* the driver, but doesn't exist
> within the build directory.
> Hence when running the library, the installation bindir needs to be in
> the PATH.  In particular, (in
> https://gcc.gnu.org/ml/jit/2014-q4/msg00005.html ) when running the jit
> testsuite we rely on the driver having been installed, and in jit.exp we
> need to temporarily prepend the installation bindir onto the front of
> PATH when running test programs linked against libgccjit.so.  Hence we
> need to know what bindir is from expect, hence we add it to site.exp.

Even if the driver's been installed, it might not be in the configured 
bindir but in some other DESTDIR.

Really, the need for an installed driver for testing should be avoided. 
The ideal way to do that is for "make check" to install to a staging 
directory within the build directory (generally, the DejaGnu approach of 
passing lots of -B etc. options to tell bits of the toolchain how to find 
each other, with lots of relevant logic hardcoded inside DejaGnu itself, 
is problematic and the staging directory approach would be better, 
although it has various other complications given that GCC needs to find 
separately built / installed pieces such as binutils and runtime 
libraries).  Of course that's a much more general issue - I suppose 
someone with an installation in a DESTDIR can still test as-is by 
specifying a different value of bindir on the "make" command line that 
builds site.exp?

-- 
Joseph S. Myers
joseph@codesourcery.com

  reply	other threads:[~2014-10-15 18:48 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-13 17:39 [PATCH 0/5] Merger of jit branch (v2) David Malcolm
2014-10-13 17:39 ` [PATCH 3/5] timevar.h: Add an auto_timevar class David Malcolm
2014-10-14  9:14   ` Richard Biener
2014-10-14 15:58     ` David Malcolm
2014-10-15  8:09       ` Richard Biener
2014-10-13 17:39 ` [PATCH 1/5] libiberty: Expose choose_tmpdir, and fix constness of return type David Malcolm
2014-10-15 17:34   ` Jeff Law
2014-10-15 19:10     ` David Malcolm
2014-10-15 19:22       ` DJ Delorie
2014-10-13 17:39 ` [PATCH 2/5] gcc: configure and Makefile changes needed by jit David Malcolm
2014-10-15 17:37   ` Jeff Law
2014-10-15 18:48     ` David Malcolm
2014-10-15 19:00       ` Joseph S. Myers [this message]
2014-10-15 21:01       ` Jeff Law
2014-10-15 21:46         ` David Malcolm
2014-10-17 16:20           ` [PATCH] Avoid the need to install when running the jit testsuite David Malcolm
2014-10-17 17:58             ` Joseph S. Myers
2014-10-20 17:59       ` [jit] Drop libgccjit.pc David Malcolm
2014-10-20 20:12         ` Basile Starynkevitch
2014-10-20 20:30           ` Matthias Klose
2014-10-20 20:44           ` David Malcolm
2014-10-13 18:38 ` [PATCH 4/5] State cleanups David Malcolm
2014-10-16 22:08   ` [PATCH 4/5] State cleanups -- also note for MPX work Jeff Law
2014-10-17  2:12     ` David Malcolm
2014-10-17 17:04       ` Jeff Law
2014-10-14 15:14 ` Patches 5-10 of jit merger (was: Re: [PATCH 0/5] Merger of jit branch (v2)) David Malcolm
2014-10-14 15:17   ` [PATCH 05/10] JIT-related changes outside of jit subdir David Malcolm
2014-10-15 17:46     ` Jeff Law
2014-10-17 21:52     ` Joseph S. Myers
2014-10-20 19:59       ` [jit] Add Sphinx to install.texi David Malcolm
2014-10-21  0:01         ` Joseph S. Myers
2014-10-21 16:20         ` Gerald Pfeifer
2014-10-21 19:30           ` David Malcolm
2014-10-30  3:08             ` [jit] Tweaks " David Malcolm
2014-10-14 15:22   ` [PATCH 06/10] Heart of the JIT implementation (was: Re: [PATCH 0/5] Merger of jit branch (v2)) David Malcolm
2014-10-17 21:54     ` Joseph S. Myers
2014-10-20 18:58       ` [jit] Error-handling within gcc::jit::dump David Malcolm
2014-10-21  0:01         ` Joseph S. Myers
2014-10-30 19:29       ` [PATCH 06/10] Heart of the JIT implementation (was: Re: [PATCH 0/5] Merger of jit branch (v2)) David Malcolm
2014-10-31  5:16         ` Joseph S. Myers
2014-10-31  6:30           ` [PATCH 06/10] Heart of the JIT implementation Jeff Law
2014-10-14 15:24   ` [PATCH 07/10] Testsuite for the JIT (Re: Patches 5-10 of jit merger (was: Re: [PATCH 0/5] Merger of jit branch (v2))) David Malcolm
2014-10-15 17:50     ` [PATCH 07/10] Testsuite for the JIT (Re: Patches 5-10 of jit merger Jeff Law
2014-10-15 20:04       ` Mike Stump
2014-10-14 15:39   ` [PATCH 10/10] ChangeLog files (Re: Patches 5-10 of jit merger (was: Re: [PATCH 0/5] Merger of jit branch (v2))) David Malcolm
2014-10-15 17:55     ` [PATCH 10/10] ChangeLog files (Re: Patches 5-10 of jit merger Jeff Law
2014-10-15 17:02   ` [PATCH 08/10] Documentation for the JIT library (Re: Patches 5-10 of jit merger) David Malcolm
2014-10-15 20:51     ` Jeff Law
2014-10-21 19:02       ` [jit] Update the docs David Malcolm
2014-10-15 17:03   ` [PATCH 09/10] Prebuilt texinfo documentation for the JIT library (Re: Patches 5-10 of jit merger) David Malcolm
2014-10-15 17:52     ` Jeff Law

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=Pine.LNX.4.64.1410151842270.11645@digraph.polyomino.org.uk \
    --to=joseph@codesourcery.com \
    --cc=dmalcolm@redhat.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=jit@gcc.gnu.org \
    --cc=law@redhat.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).