Bug#711558: PR52306 (was Re: Bug#711558: gcc-4.8: [m68k] patch set 2)
On Thu, 22 Aug 2013 22:26:51 +0000 (UTC), Thorsten Glaser <tg@mirbsd.de> wrote:
> Matthias Klose dixit:
>
> >yes, I do reject this.
>
> I see. Would you please=E2=80=A6
>
> >> =E2=80=9Cfor the time being=E2=80=9D? If so, would you accept a patch
> >> that just disables -fauto-inc-dec on m68k *always*,
> >> even in the cases where it doesn=E2=80=99t 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
> =20
> 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
> =20
> ; -fcheck-bounds causes gcc to generate array bounds checks.
>
> Or maybe this one (although it=E2=80=99s got the malus that it can=E2=80=99=
> 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 =3D 0;
> flag_modulo_sched =3D 0;
> }
> + /* PR52306 */
> + flag_auto_inc_dec =3D 0;
> }
> =20
> /* Generate a macro of the form __mPREFIX_cpu_NAME, where PREFIX is the
Tweaking the option so that it defaults to OFF for m68k, but still can be
enabled, would be preferable. I haven't looked at how to achieve that in
gcc's options machinery.
/Mikael
Reply to: