Bug#711558: PR52306 (was Re: Bug#711558: gcc-4.8: [m68k] patch set 2)

[Thorsten Glaser <tg@mirbsd.de>]

Matthias Klose dixit:

>yes, I do reject this.

I see. Would you please…

>> “for the time being”? If so, would you accept a patch
>> that just disables -fauto-inc-dec on m68k *always*,
>> even in the cases where it doesn’t ICE? (one-liner)

answer whether this would be considerable? (Untested,
but should have the desired effect, right Mikael?)

--- a/src/gcc/common.opt
+++ b/src/gcc/common.opt
@@ -858,7 +858,7 @@ Common Report Var(flag_asynchronous_unwi
 Generate unwind tables that are exact at each instruction boundary
 
 fauto-inc-dec
-Common Report Var(flag_auto_inc_dec) Init(1)
+Common Report Var(flag_auto_inc_dec) Init(0)
 Generate auto-inc/dec instructions
 
 ; -fcheck-bounds causes gcc to generate array bounds checks.

Or maybe this one (although it’s got the malus that it can’t
be re-enabled for testing):

--- a/src/gcc/config/m68k/m68k.c
+++ b/src/gcc/config/m68k/m68k.c
@@ -663,6 +663,8 @@ m68k_override_options_after_change (void
       flag_schedule_insns_after_reload = 0;
       flag_modulo_sched = 0;
     }
+  /* PR52306 */
+  flag_auto_inc_dec = 0;
 }
 
 /* Generate a macro of the form __mPREFIX_cpu_NAME, where PREFIX is the

>not many thanks for trying to sneak in this into the
>Debian package.

I’m just forwarding what we use in production. This is
really necessary right now to work around bugs with the
GNU Chagrin Collection.

Apparently just always disabling optimisation is the
way, not just when it is problematic…

bye,
//mirabilos
-- 
In traditional syntax ' is ignored, but in c99 everything between two ' is
handled as character constant.  Therefore you cannot use ' in a preproces-
sing file in c99 mode.	-- Ragge
No faith left in ISO C99, undefined behaviour, etc.