> -----Original Message-----
> From: Iain Sandoe [mailto:i...@codesourcery.com]
> Sent: Friday, November 8, 2013 3:28 PM
> To: Iyer, Balaji V
> Cc: Bernhard Reutner-Fischer; Jakub Jelinek; Joseph S. Myers; Tobias Burnus;
> gcc patches; Mike Stump
> Subject: Re: [PATCH] Testsuite / Cilk Plus: Include library path in compile 
> flags
> in gcc.dg/cilk-plus/cilk-plus.exp
> 
> hi Balaji,
> 
> On 8 Nov 2013, at 10:01, Mike Stump <mikest...@comcast.net> wrote:
> 
> > On Nov 8, 2013, at 4:09 AM, Bernhard Reutner-Fischer
> <rep.dot....@gmail.com> wrote:
> >> On 6 November 2013 08:04, Jakub Jelinek <ja...@redhat.com> wrote:
> >>> On Wed, Nov 06, 2013 at 02:24:01AM +0000, Iyer, Balaji V wrote:
> >>>>   Fixed patch is attached. The responses to your question are given
> below. Is this patch OK?
> >
> >> May i suggest you rephrase the .exp so it does not line-wrap and is
> >> actually readable like in attached (untested) 01 or at least 00?
> >
> > I like the direction irrespective of line length.  :-)
> 
> 
> the  current cilk-plus c testsuite  is now much more realistic in run-time
> (2:40mins vs. 160mins on my laptop)
> however:
> 
> (i)  it still fails to call " set_ld_library_path_env_vars " and thus fails 
> to work
> properly on Darwin as described in a previous mail.
> 

I am planning to look into this next week and as soon as I get  a solution, I 
will send out a patch.

> (ii) it still does not work for installed testing viz:
> 
> runtest --tool gcc --target_board=xxxx cilk-plus.exp (with some suitable xxxx
> and site.exp for your setup).
> 
I will look into this also.

> (iii) I remain surprised (as perhaps was Jeff) that existing torture 
> facilities
> were not the best solution for this.
> 
> cheers
> Iain
> 
> 

Thanks,

Balaji V. Iyer.

Reply via email to