> -----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.