An unreviewed m68k patch

Roman Zippel zippel@linux-m68k.org
Sat Sep 8 01:19:00 GMT 2007


Hi,

On Fri, 7 Sep 2007, Kazu Hirata wrote:

>   [patch] m68k: Use ASSEMBLER_DIALECT
>   http://gcc.gnu.org/ml/gcc-patches/2007-08/msg01757.html
> 
> There was a lot of exchanges about removing one syntax, but I don't
> think that's going to happen any time soon.
> 
> There was one technical suggestion from Roman Zippel
> 
>   http://gcc.gnu.org/ml/gcc-patches/2007-08/msg01849.html
> 
> but points that are raised are not bugs in the patch, so I think they
> can be addressed in a separate patch.

The problem is these patches conflict, I'll post an updated version of 
both patches as another reply). Both have been tested multiple times on 
m68k-linux/i686-linux.
I very much prefer to cleanup the jump instructions properly. To address 
some more possible concerns. As can be seen by this:

$ grep MOTOROLA gcc/config.gcc
        tm_defines="${tm_defines} MOTOROLA=1 USE_GAS"
        tm_defines="${tm_defines} MOTOROLA=1 USE_GAS"
        tm_defines="${tm_defines} MOTOROLA=1 USE_GAS"
        tm_defines="${tm_defines} MOTOROLA=1 USE_GAS"
        tm_defines="${tm_defines} MOTOROLA=1 USE_GAS UCLIBC_DEFAULT=1"
        tm_defines="${tm_defines} MOTOROLA=1 USE_GAS"
        tm_defines="${tm_defines} MOTOROLA=1 USE_GAS"

there is assumptation that targets that use the Motorola syntax also use 
gas.
Also the first patch has been for quite some time already without any 
problems in Debian.

bye, Roman



More information about the Gcc-patches mailing list