* [Bug driver/41024] driver passes wrong parameter to as
2009-08-10 11:38 [Bug driver/41024] New: driver passes wrong parameter to as rainer at emrich-ebersheim dot de
@ 2009-09-03 14:41 ` fxcoudert at gcc dot gnu dot org
2009-09-05 10:02 ` ktietz at gcc dot gnu dot org
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: fxcoudert at gcc dot gnu dot org @ 2009-09-03 14:41 UTC (permalink / raw)
To: gcc-bugs
------- Comment #1 from fxcoudert at gcc dot gnu dot org 2009-09-03 14:40 -------
It's because of this in gcc/config/i386/mingw-w64.h:
#define ASM_SPEC "%{v:-V} %{n} %{T} %{Ym,*} %{Yd,*} \
%{Wa,*:%*} %{m32:--32} %{m64:--64}"
The "%{v:-V}" part is what's triggering what you see. Now, the question is why
is your as rejecting the -V option? My linux as accepts it, my darwin as also,
but i586-pc-mingw32-as doesn't...
After further investigation, it seems that handling the option -V on i386
processors is only done for ELF targets; can't even begin to think of a good
reason for that, but that's what gas source code says. So, I recommend the
following (untested) patch to GCC:
Index: gcc/config/i386/mingw-w64.h
===================================================================
--- gcc/config/i386/mingw-w64.h (revision 151373)
+++ gcc/config/i386/mingw-w64.h (working copy)
@@ -36,7 +36,7 @@
/* Enable multilib. */
#undef ASM_SPEC
-#define ASM_SPEC "%{v:-V} %{n} %{T} %{Ym,*} %{Yd,*} \
+#define ASM_SPEC "%{v} %{n} %{T} %{Ym,*} %{Yd,*} \
%{Wa,*:%*} %{m32:--32} %{m64:--64}"
#if TARGET_64BIT_DEFAULT
--
fxcoudert at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |fxcoudert at gcc dot gnu dot
| |org
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Keywords| |patch
Last reconfirmed|0000-00-00 00:00:00 |2009-09-03 14:40:55
date| |
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41024
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug driver/41024] driver passes wrong parameter to as
2009-08-10 11:38 [Bug driver/41024] New: driver passes wrong parameter to as rainer at emrich-ebersheim dot de
2009-09-03 14:41 ` [Bug driver/41024] " fxcoudert at gcc dot gnu dot org
@ 2009-09-05 10:02 ` ktietz at gcc dot gnu dot org
2009-09-05 10:14 ` fxcoudert at gcc dot gnu dot org
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: ktietz at gcc dot gnu dot org @ 2009-09-05 10:02 UTC (permalink / raw)
To: gcc-bugs
------- Comment #2 from ktietz at gcc dot gnu dot org 2009-09-05 10:02 -------
(In reply to comment #1)
> It's because of this in gcc/config/i386/mingw-w64.h:
>
> #define ASM_SPEC "%{v:-V} %{n} %{T} %{Ym,*} %{Yd,*} \
> %{Wa,*:%*} %{m32:--32} %{m64:--64}"
>
>
> The "%{v:-V}" part is what's triggering what you see. Now, the question is why
> is your as rejecting the -V option? My linux as accepts it, my darwin as also,
> but i586-pc-mingw32-as doesn't...
>
> After further investigation, it seems that handling the option -V on i386
> processors is only done for ELF targets; can't even begin to think of a good
> reason for that, but that's what gas source code says. So, I recommend the
> following (untested) patch to GCC:
>
> Index: gcc/config/i386/mingw-w64.h
> ===================================================================
> --- gcc/config/i386/mingw-w64.h (revision 151373)
> +++ gcc/config/i386/mingw-w64.h (working copy)
> @@ -36,7 +36,7 @@
> /* Enable multilib. */
>
> #undef ASM_SPEC
> -#define ASM_SPEC "%{v:-V} %{n} %{T} %{Ym,*} %{Yd,*} \
> +#define ASM_SPEC "%{v} %{n} %{T} %{Ym,*} %{Yd,*} \
> %{Wa,*:%*} %{m32:--32} %{m64:--64}"
>
> #if TARGET_64BIT_DEFAULT
>
Ok, could you sent a patch for it. It is pre-approved by me.
Thanks,
Kai
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41024
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug driver/41024] driver passes wrong parameter to as
2009-08-10 11:38 [Bug driver/41024] New: driver passes wrong parameter to as rainer at emrich-ebersheim dot de
2009-09-03 14:41 ` [Bug driver/41024] " fxcoudert at gcc dot gnu dot org
2009-09-05 10:02 ` ktietz at gcc dot gnu dot org
@ 2009-09-05 10:14 ` fxcoudert at gcc dot gnu dot org
2009-09-05 11:20 ` ktietz at gcc dot gnu dot org
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: fxcoudert at gcc dot gnu dot org @ 2009-09-05 10:14 UTC (permalink / raw)
To: gcc-bugs
------- Comment #3 from fxcoudert at gcc dot gnu dot org 2009-09-05 10:14 -------
(In reply to comment #2)
> Ok, could you sent a patch for it. It is pre-approved by me.
I can't test it, have you?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41024
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug driver/41024] driver passes wrong parameter to as
2009-08-10 11:38 [Bug driver/41024] New: driver passes wrong parameter to as rainer at emrich-ebersheim dot de
` (2 preceding siblings ...)
2009-09-05 10:14 ` fxcoudert at gcc dot gnu dot org
@ 2009-09-05 11:20 ` ktietz at gcc dot gnu dot org
2009-09-05 13:14 ` ktietz at gcc dot gnu dot org
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: ktietz at gcc dot gnu dot org @ 2009-09-05 11:20 UTC (permalink / raw)
To: gcc-bugs
------- Comment #4 from ktietz at gcc dot gnu dot org 2009-09-05 11:20 -------
(In reply to comment #3)
> (In reply to comment #2)
> > Ok, could you sent a patch for it. It is pre-approved by me.
>
> I can't test it, have you?
>
I am retesting it. I think {v:-v} is better here.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41024
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug driver/41024] driver passes wrong parameter to as
2009-08-10 11:38 [Bug driver/41024] New: driver passes wrong parameter to as rainer at emrich-ebersheim dot de
` (3 preceding siblings ...)
2009-09-05 11:20 ` ktietz at gcc dot gnu dot org
@ 2009-09-05 13:14 ` ktietz at gcc dot gnu dot org
2009-09-05 14:14 ` fxcoudert at gcc dot gnu dot org
2009-09-05 14:17 ` [Bug target/41024] " fxcoudert at gcc dot gnu dot org
6 siblings, 0 replies; 8+ messages in thread
From: ktietz at gcc dot gnu dot org @ 2009-09-05 13:14 UTC (permalink / raw)
To: gcc-bugs
------- Comment #5 from ktietz at gcc dot gnu dot org 2009-09-05 13:14 -------
> I am retesting it. I think {v:-v} is better here.
This patch works fine. Could you post a patch for it. It is pre-approved.
Thanks,
Kai
Index: mingw-w64.h
===================================================================
--- mingw-w64.h (revision 151421)
+++ mingw-w64.h (working copy)
@@ -36,7 +36,7 @@
/* Enable multilib. */
#undef ASM_SPEC
-#define ASM_SPEC "%{v:-V} %{n} %{T} %{Ym,*} %{Yd,*} \
+#define ASM_SPEC "%{v:-v} %{n} %{T} %{Ym,*} %{Yd,*} \
%{Wa,*:%*} %{m32:--32} %{m64:--64}"
#if TARGET_64BIT_DEFAULT
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41024
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug driver/41024] driver passes wrong parameter to as
2009-08-10 11:38 [Bug driver/41024] New: driver passes wrong parameter to as rainer at emrich-ebersheim dot de
` (4 preceding siblings ...)
2009-09-05 13:14 ` ktietz at gcc dot gnu dot org
@ 2009-09-05 14:14 ` fxcoudert at gcc dot gnu dot org
2009-09-05 14:17 ` [Bug target/41024] " fxcoudert at gcc dot gnu dot org
6 siblings, 0 replies; 8+ messages in thread
From: fxcoudert at gcc dot gnu dot org @ 2009-09-05 14:14 UTC (permalink / raw)
To: gcc-bugs
------- Comment #6 from fxcoudert at gcc dot gnu dot org 2009-09-05 14:14 -------
Subject: Bug 41024
Author: fxcoudert
Date: Sat Sep 5 14:14:28 2009
New Revision: 151450
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=151450
Log:
PR target/41024
* config/i386/mingw-w64.h (ASM_SPEC): Pass -v instead of -V to
the assembler.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/i386/mingw-w64.h
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41024
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug target/41024] driver passes wrong parameter to as
2009-08-10 11:38 [Bug driver/41024] New: driver passes wrong parameter to as rainer at emrich-ebersheim dot de
` (5 preceding siblings ...)
2009-09-05 14:14 ` fxcoudert at gcc dot gnu dot org
@ 2009-09-05 14:17 ` fxcoudert at gcc dot gnu dot org
6 siblings, 0 replies; 8+ messages in thread
From: fxcoudert at gcc dot gnu dot org @ 2009-09-05 14:17 UTC (permalink / raw)
To: gcc-bugs
------- Comment #7 from fxcoudert at gcc dot gnu dot org 2009-09-05 14:16 -------
Fixed on trunk, does not exist on 4.4 branch.
--
fxcoudert at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Component|driver |target
Resolution| |FIXED
Target Milestone|--- |4.5.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41024
^ permalink raw reply [flat|nested] 8+ messages in thread