[Bug target/26885] New: [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[Bug target/26885] New: [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[jakub at gcc dot gnu dot org]

Up until
2005-06-13  Mark Mitchell  <mark@codesourcery.com>

        * config/i386/x86-64.h (ASM_SPEC): Explicitly pass --64 to the
        assembler in 64-bit mode.
change -m64 -m32 used to work on x86_64 (I know it is quite fragile and
on many arches doesn't DTRT), but now as the asm spec has:
%{m32:--32} %{-m64:--64} we for both -m64 -m32 and -m32 -m64 end up with
--32 --64 being passed to the assembler (in that order).

Guess best would be to say in *.opt rather than RejectNegative what the
negative
of the option is (for m32 Negative(m64) and for m64 Negative(m32) or something
like that) and handle this in gcc.c/opts.c.


-- 
           Summary: [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit
                    object
           Product: gcc
           Version: 4.1.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: jakub at gcc dot gnu dot org
GCC target triplet: x86_64-linux


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2006-03-27 16:37 -------
Confirmed, -m32 -m64 is also broken too.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pinskia at gcc dot gnu dot
                   |                            |org
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2006-03-27 16:37:46
               date|                            |
   Target Milestone|---                         |4.1.1


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[jakub at gcc dot gnu dot org]

------- Comment #2 from jakub at gcc dot gnu dot org  2006-03-28 08:20 -------
On x86-64 Linux?  -m32 -m64 works just fine, for cc1 obviously (-m64 wins)
and for gas as well (--32 --64 is passed, the last one wins).
The reason why -m64 -m32 doesn't work is that for cc1 -m32 wins and for
gas we pass --32 --64 for that switch combination too, which means as is
trying to assemble the 32-bit asm as 64-bit assembly.
This breaks Linux kernel build (which has -m64 in AFLAGS and then appends
-m32 for the few files that need to be built with 32-bit assembler and linked
into 32-bit vDSO).


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[mmitchel at gcc dot gnu dot org]

------- Comment #3 from mmitchel at gcc dot gnu dot org  2006-03-31 02:52 -------
Jakub --

Sorry about introducing this bug!  I'm not sure how best to handle it, but I'll
fix it somehow, if you like.  I won't have time to work on it for about a week,
but I'll be happy to handle it then.  If you'd like that, please assign the bug
to me.

Thanks,

-- Mark


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[jakub at gcc dot gnu dot org]

-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |mmitchel at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[pinskia at gcc dot gnu dot org]

------- Comment #4 from pinskia at gcc dot gnu dot org  2006-04-22 00:43 -------
*** Bug 27253 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hjl at lucon dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[pinskia at physics dot uc dot edu]

------- Comment #5 from pinskia at physics dot uc dot edu  2006-04-22 01:16 -------
Lets try this. (just a test).


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[hjl at lucon dot org]

------- Comment #6 from hjl at lucon dot org  2006-04-22 15:33 -------
[hjl@gnu-9 stage1-gcc]$ ./xgcc -B./ -c -g -v  /tmp/x.s  -gstabs -gdwarf-2
Reading specs from ./specs
Target: x86_64-unknown-linux-gnu
Configured with: /net/gnu-13/export/gnu/src/gcc/gcc/configure
--enable-clocale=gnu --with-system-zlib --with-demangler-in-ld --enable-shared
--enable-threads=posix --enable-haifa --enable-checking=assert
--prefix=/usr/gcc-4.2 --with-local-prefix=/usr/local
Thread model: posix
gcc version 4.2.0 20060421 (experimental) [trunk revision 113135 clean]
 ./as --gstabs -V -Qy -o x.o /tmp/x.s
GNU assembler version 2.17.50.0.1 (x86_64-unknown-linux-gnu) using BFD version
2.17.50.0.1 20060417


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[pinskia at gcc dot gnu dot org]

------- Comment #7 from pinskia at gcc dot gnu dot org  2006-04-23 02:24 -------
There is a patch posted:
http://gcc.gnu.org/ml/gcc-patches/2006-04/msg00857.html

But I feel it is the wrong approach in that the driver now needs to know about
the options which is semi wrong.  There must be an easier way to fix that what
is done.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[mmitchel at gcc dot gnu dot org]

------- Comment #8 from mmitchel at gcc dot gnu dot org  2006-05-16 05:52 -------
This version of HJ's patch:

http://gcc.gnu.org/ml/gcc-patches/2006-04/msg01013.html

is OK for mainline and 4.1.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[hjl at gcc dot gnu dot org]

------- Comment #9 from hjl at gcc dot gnu dot org  2006-05-16 14:27 -------
Subject: Bug 26885

Author: hjl
Date: Tue May 16 14:27:18 2006
New Revision: 113824

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=113824
Log:
gcc/

2006-05-16  H.J. Lu  <hongjiu.lu@intel.com>

        PR driver/26885
        * Makefile.in (GCC_OBJS): New.
        (OBJS-common): Add opts-common.o.
        (xgcc$(exeext)): Replace gcc.o with $(GCC_OBJS).
        (cpp$(exeext)): Likewise.
        (gcc.o): Also depend on opts.h.
        (opts-common.o): New.

        * common.opt (gcoff): Add Negative(gdwarf-2).
        (gdwarf-2): Add Negative(gstabs).
        (gstabs): Add Negative(gstabs+).
        (gstabs+): Add Negative(gvms).
        (gvms): Add Negative(gxcoff).
        (gxcoff): Add Negative(gxcoff+).
        (gxcoff+): Add Negative(gcoff).
        * config/i386/i386.opt (m32): Add Negative(m64).
        (m64): Add Negative(m32).

        * doc/options.texi: Document the Negative option.

        * gcc.c: Include "opts.h".
        (main): Call prune_options after expandargv.

        * optc-gen.awk: Generate common declarations for all flag
        variables in options.c. Output the neg_index field.

        * opts.c (find_opt): Moved to ...
        * opts-common.c: Here. New file.

        * opts.h (cl_option): Add a neg_index field.
        (find_opt): New.
        (prune_options): Likewise.

gcc/cp/

2006-05-16  H.J. Lu  <hongjiu.lu@intel.com>

        PR driver/26885
        * Make-lang.in (GXX_OBJS): Replace gcc.o with $(GCC_OBJS).

gcc/fortran/

2006-05-16  H.J. Lu  <hongjiu.lu@intel.com>

        PR driver/26885
        * Make-lang.in (GFORTRAN_D_OBJS): Replace gcc.o with
        $(GCC_OBJS).

gcc/java/

2006-05-16  H.J. Lu  <hongjiu.lu@intel.com>

        PR driver/26885
        * Make-lang.in ($(GCJ)$(exeext)): Replace gcc.o with
        $(GCC_OBJS).

gcc/treelang/

2006-05-16  H.J. Lu  <hongjiu.lu@intel.com>

        PR driver/26885
        * Make-lang.in (gtreelang$(exeext)): Replace gcc.o with
        $(GCC_OBJS).

Added:
    trunk/gcc/opts-common.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/Makefile.in
    trunk/gcc/common.opt
    trunk/gcc/config/i386/i386.opt
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/Make-lang.in
    trunk/gcc/doc/options.texi
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/Make-lang.in
    trunk/gcc/gcc.c
    trunk/gcc/java/ChangeLog
    trunk/gcc/java/Make-lang.in
    trunk/gcc/optc-gen.awk
    trunk/gcc/opts.c
    trunk/gcc/opts.h
    trunk/gcc/treelang/ChangeLog
    trunk/gcc/treelang/Make-lang.in


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[hjl at lucon dot org]

------- Comment #10 from hjl at lucon dot org  2006-05-16 16:32 -------
Hi Mark,

I realized that I had an updated patch

http://gcc.gnu.org/ml/gcc-patches/2006-05/msg00090.html

to address a concern from

http://gcc.gnu.org/ml/gcc-patches/2006-05/msg00050.html

Can I update gcc/doc/invoke.texi?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[mark at codesourcery dot com]

------- Comment #11 from mark at codesourcery dot com  2006-05-16 16:51 -------
Subject: Re:  [4.1/4.2 regression] -m64 -m32 no longer creates
 32-bit object

hjl at lucon dot org wrote:
> ------- Comment #10 from hjl at lucon dot org  2006-05-16 16:32 -------
> Hi Mark,
> 
> I realized that I had an updated patch
> 
> http://gcc.gnu.org/ml/gcc-patches/2006-05/msg00090.html
> 
> to address a concern from
> 
> http://gcc.gnu.org/ml/gcc-patches/2006-05/msg00050.html
> 
> Can I update gcc/doc/invoke.texi?

OK.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[hjl at gcc dot gnu dot org]

------- Comment #12 from hjl at gcc dot gnu dot org  2006-05-16 17:06 -------
Subject: Bug 26885

Author: hjl
Date: Tue May 16 17:06:05 2006
New Revision: 113828

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=113828
Log:
gcc/

2006-05-16  H.J. Lu  <hongjiu.lu@intel.com>

        PR driver/26885
        * Makefile.in (GCC_OBJS): New.
        (OBJS-common): Add opts-common.o.
        (xgcc$(exeext)): Replace gcc.o with $(GCC_OBJS).
        (cpp$(exeext)): Likewise.
        (gcc.o): Also depend on opts.h.
        (opts-common.o): New.

        * common.opt (gcoff): Add Negative(gdwarf-2).
        (gdwarf-2): Add Negative(gstabs).
        (gstabs): Add Negative(gstabs+).
        (gstabs+): Add Negative(gvms).
        (gvms): Add Negative(gxcoff).
        (gxcoff): Add Negative(gxcoff+).
        (gxcoff+): Add Negative(gcoff).
        * config/i386/i386.opt (m32): Add Negative(m64).
        (m64): Add Negative(m32).

        * doc/options.texi: Document the Negative option.

        * gcc.c: Include "opts.h".
        (main): Call prune_options after expandargv.

        * optc-gen.awk: Generate common declarations for all flag
        variables in options.c. Output the neg_index field.

        * opts.c (find_opt): Moved to ...
        * opts-common.c: Here. New file.

        * opts.h (cl_option): Add a neg_index field.
        (find_opt): New.
        (prune_options): Likewise.

gcc/cp/

2006-05-16  H.J. Lu  <hongjiu.lu@intel.com>

        PR driver/26885
        * Make-lang.in (GXX_OBJS): Replace gcc.o with $(GCC_OBJS).

gcc/fortran/

2006-05-16  H.J. Lu  <hongjiu.lu@intel.com>

        PR driver/26885
        * Make-lang.in (GFORTRAN_D_OBJS): Replace gcc.o with
        $(GCC_OBJS).

gcc/java/

2006-05-16  H.J. Lu  <hongjiu.lu@intel.com>

        PR driver/26885
        * Make-lang.in ($(GCJ)$(exeext)): Replace gcc.o with
        $(GCC_OBJS).

gcc/treelang/

2006-05-16  H.J. Lu  <hongjiu.lu@intel.com>

        PR driver/26885
        * Make-lang.in (gtreelang$(exeext)): Replace gcc.o with
        $(GCC_OBJS).

Added:
    branches/gcc-4_1-branch/gcc/opts-common.c
Modified:
    branches/gcc-4_1-branch/gcc/ChangeLog
    branches/gcc-4_1-branch/gcc/Makefile.in
    branches/gcc-4_1-branch/gcc/common.opt
    branches/gcc-4_1-branch/gcc/config/i386/i386.opt
    branches/gcc-4_1-branch/gcc/cp/ChangeLog
    branches/gcc-4_1-branch/gcc/cp/Make-lang.in
    branches/gcc-4_1-branch/gcc/doc/options.texi
    branches/gcc-4_1-branch/gcc/fortran/ChangeLog
    branches/gcc-4_1-branch/gcc/fortran/Make-lang.in
    branches/gcc-4_1-branch/gcc/gcc.c
    branches/gcc-4_1-branch/gcc/java/ChangeLog
    branches/gcc-4_1-branch/gcc/java/Make-lang.in
    branches/gcc-4_1-branch/gcc/optc-gen.awk
    branches/gcc-4_1-branch/gcc/opts.c
    branches/gcc-4_1-branch/gcc/opts.h
    branches/gcc-4_1-branch/gcc/treelang/ChangeLog
    branches/gcc-4_1-branch/gcc/treelang/Make-lang.in


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[pinskia at gcc dot gnu dot org]

------- Comment #13 from pinskia at gcc dot gnu dot org  2006-05-16 17:29 -------
Grrr, this broke bootstrap every where.  Did you test this at all?  -fno-common
added done by defualt to prevent cases like this.

-- Pinski


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[hjl at lucon dot org]

------- Comment #14 from hjl at lucon dot org  2006-05-16 17:37 -------
I didn't see -fno-common in my build logs on Linux/x86, Linux/x86-64
and Linux/ia64. I am using:

../configure \
         \
        --enable-clocale=gnu --with-system-zlib --with-demangler-in-ld \
         \
        --enable-shared \
        --enable-threads=posix \
        --enable-haifa \
        --enable-checking=assert \
        --prefix=/usr/gcc-4.2 \
        --with-local-prefix=/usr/local

to configure.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[pinskia at gcc dot gnu dot org]

------- Comment #15 from pinskia at gcc dot gnu dot org  2006-05-16 17:39 -------
(In reply to comment #14)
>         --enable-checking=assert \

There is your issue.  Why are you disabling all checking except for assert on
the mainline for testing?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[bernds_cb1 at t-online dot de]

------- Comment #16 from bernds_cb1 at t-online dot de  2006-05-16 21:30 -------
I saw the fallout of this on the mailing list, but was the patch ever sent to
gcc-patches before it was committed?


-- 

bernds_cb1 at t-online dot de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bernds_cb1 at t-online dot
                   |                            |de


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[mmitchel at gcc dot gnu dot org]

------- Comment #17 from mmitchel at gcc dot gnu dot org  2006-05-16 21:46 -------
Yes, the patch was posted; see Comment #8.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[mmitchel at gcc dot gnu dot org]

------- Comment #18 from mmitchel at gcc dot gnu dot org  2006-05-17 15:36 -------
It appears that HJ's patches are now checked in.  Can we close this PR?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1/4.2 regression] -m64 -m32 no longer creates 32-bit object

[mmitchel at gcc dot gnu dot org]

------- Comment #19 from mmitchel at gcc dot gnu dot org  2006-05-25 02:31 -------
Will not be fixed in 4.1.1; adjust target milestone to 4.1.2.


-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.1.1                       |4.1.2


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1 regression] -m64 -m32 no longer creates 32-bit object

[pinskia at gcc dot gnu dot org]

------- Comment #20 from pinskia at gcc dot gnu dot org  2006-07-05 09:05 -------
Fixed on the mainline.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.1/4.2 regression] -m64 - |[4.1 regression] -m64 -m32
                   |m32 no longer creates 32-bit|no longer creates 32-bit
                   |object                      |object


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1 regression] -m64 -m32 no longer creates 32-bit object

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.1.2                       |4.1.3


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885

[Bug target/26885] [4.1 regression] -m64 -m32 no longer creates 32-bit object

[jsm28 at gcc dot gnu dot org]

------- Comment #21 from jsm28 at gcc dot gnu dot org  2008-07-04 15:24 -------
Closing 4.1 branch; logs indicate this was fixed in 4.1.2.


-- 

jsm28 at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|4.1.3                       |4.1.2


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26885