[Patch] compile libgomp.c++/pr30703.C with -shared-libgcc

Mike Stump mrs@apple.com
Thu Mar 22 19:01:00 GMT 2007


Ian, I was wondering what your take on this as a shared libgcc  
maintainer?  I think it borders on obvious, but there is some concern  
that -shared-libgcc might not somehow work on some platforms.

Ok?

On Mar 15, 2007, at 12:47 PM, Mike Stump wrote:

> On Mar 15, 2007, at 6:50 AM, Jack Howarth wrote:
>>    The attached patch forces the libgomp.c++/pr30703 testsuite test
>> to compile with the shared libgcc to prevent false failures on  
>> platforms
>> like Darwin. This is required because the dejagnu harness used for
>> the libgomp.c++ tests doesn't compile those tests with the g++
>> compiler. Okay for trunk and 4.2 branch?
>
> I reviewed the patch and the gcc documentation and it looks correct  
> enough to add that flag.  I think a better way to fix this would be  
> to instead do:
>
> Doing diffs in libgomp:
> --- libgomp/testsuite/libgomp.c++/c++.exp.~1~	2006-11-16  
> 23:18:00.000000000 -0800
> +++ libgomp/testsuite/libgomp.c++/c++.exp	2007-03-15  
> 12:42:02.000000000 -0700
> @@ -1,6 +1,6 @@
>  set lang_library_path	"../libstdc++-v3/src/.libs"
>  set lang_test_file	"${lang_library_path}/libstdc++.a"
> -set lang_link_flags	"-lstdc++"
> +set lang_link_flags	"-shared-libgcc -lstdc++"
>
>  load_lib libgomp-dg.exp
>
> --------------
>
> The idea is to put the fix next to the thing that is broken, that  
> way it is impossible for another case to break.  Someone want to  
> agree with my patch and reasoning?  If so, I'll Ok that version.



More information about the Gcc-patches mailing list