Re: [PATCH, go] Add -mieee to various go.test tests

2012-01-20 Thread Ian Lance Taylor
Tom Tromey writes: >> "Ian" == Ian Lance Taylor writes: > > Ian> And I just have to repeat that this patch is an ugly ugly hack, since > Ian> -mieee should be the default. Perhaps we should investigate having > Ian> gcc/go/gospec.c or gcc/go/lang-specs.h somehow add -mieee for those > Ian>

Re: [PATCH, go] Add -mieee to various go.test tests

2012-01-20 Thread Tom Tromey
> "Ian" == Ian Lance Taylor writes: Ian> And I just have to repeat that this patch is an ugly ugly hack, since Ian> -mieee should be the default. Perhaps we should investigate having Ian> gcc/go/gospec.c or gcc/go/lang-specs.h somehow add -mieee for those Ian> targets which require it. For

Re: [PATCH, go] Add -mieee to various go.test tests

2012-01-20 Thread Ian Lance Taylor
Uros Bizjak writes: > OK, I agree. But please note that the patch also includes a fix to > pass correctly formatted options to go_target_compile. I guess this > part should be installed anyway, otherwise go_target_compile simply > ignores passed extra options. Good point, that part of the patch

Re: [PATCH, go] Add -mieee to various go.test tests

2012-01-20 Thread Uros Bizjak
On Fri, Jan 20, 2012 at 12:39 AM, Ian Lance Taylor wrote: >> Attached patch adds -mieee to tests that need full IEEE compliance to >> pass. While working on patch, I have noticed that go-test.exp doesn't >> pass DEFAULT_GOCFLAGS to go_target_compile procedure in expected >> format (so, these simp

Re: [PATCH, go] Add -mieee to various go.test tests

2012-01-19 Thread Mike Stump
On Jan 19, 2012, at 3:39 PM, Ian Lance Taylor wrote: > And I just have to repeat that this patch is an ugly ugly hack, since > -mieee should be the default. Perhaps we should investigate having > gcc/go/gospec.c or gcc/go/lang-specs.h somehow add -mieee for those > targets which require it. I agr

Re: [PATCH, go] Add -mieee to various go.test tests

2012-01-19 Thread Ian Lance Taylor
Uros Bizjak writes: > Attached patch adds -mieee to tests that need full IEEE compliance to > pass. While working on patch, I have noticed that go-test.exp doesn't > pass DEFAULT_GOCFLAGS to go_target_compile procedure in expected > format (so, these simply get ignored). With this issue fixed, we

[PATCH, go] Add -mieee to various go.test tests

2012-01-19 Thread Uros Bizjak
Hello! Attached patch adds -mieee to tests that need full IEEE compliance to pass. While working on patch, I have noticed that go-test.exp doesn't pass DEFAULT_GOCFLAGS to go_target_compile procedure in expected format (so, these simply get ignored). With this issue fixed, we can add -mieee to DEF