This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [testsuite, build] Convert boehm-gc testsuite to DejaGnu (PR boehm-gc/11412)


Mike Stump <mikestump@comcast.net> writes:

> On Jan 24, 2011, at 6:01 AM, Rainer Orth wrote:
>> +    # FIXME: Explain options.
>> +    set shopt "-version-info 1:2:0 -no-undefined -rpath /nowhere"
>> +    # Prefix -shared-libgcc with -Wc, to get it through libtool.
>> +    lappend shopt -Wc,-shared-libgcc
>
> Seems not very portable...  though, maybe it really is.

I has to be used for a couple of target libraries already, so using it
in the testsuite doesn't make things worse (provided we're allowed to
assume that the testsuite are only run with gcc, not vendor compilers).

>> +    catch { exec $libtool --mode=clean rm -f $basename.lo }
>
> If libtool runs on the host, this doesn't do that, try:
>
>   [remote_exec host ... ]
>
> see gcc.exp for examples.
>
>> +	catch { exec $libtool --mode=clean rm -f [file rootname [file tail $testcase]] $shlib }
>
> Likewise.

Ok, will do.

> Other than that, looks reasonable.  Though, I'd probably say, stage1, as I said earlier.  If you want it for the current release, we can also ask a RM to weigh in...  I'd be fine with them preferring it sooner, if they want.

I don't think the benefit outweighs the risk: we've lived with a non-dg
testsuite for years (even the PR is 7.5 years old), so I see no need to
hurry. 

	Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]