https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61320

--- Comment #62 from ro at CeBiTec dot Uni-Bielefeld.DE <ro at CeBiTec dot 
Uni-Bielefeld.DE> ---
> --- Comment #60 from Richard Biener <rguenth at gcc dot gnu.org> ---
[...]
> Fix:
>
> Index: gcc/tree-ssa-loop-ivopts.c
> ===================================================================
> --- gcc/tree-ssa-loop-ivopts.c  (revision 213050)
> +++ gcc/tree-ssa-loop-ivopts.c  (working copy)
> @@ -1704,6 +1704,8 @@ may_be_unaligned_p (tree ref, tree step)
>      return false;
>
>    unsigned int align = TYPE_ALIGN (TREE_TYPE (ref));
> +  if (GET_MODE_ALIGNMENT (TYPE_MODE (TREE_TYPE (ref))) > align)
> +    align = GET_MODE_ALIGNMENT (TYPE_MODE (TREE_TYPE (ref)));
>
>    unsigned HOST_WIDE_INT bitpos;
>    unsigned int ref_align;
>
> can you test and apply that patch?

With the patch, SPARC bootstrap concluded on mainline.  I'm seeing two
different regressions with this patch in tree:

* on i386-pc-solaris2.10, gcc.dg/pch/import-[12].c FAILs (SEGV in cc1
  due to unaligned movaps destination)

* on sparc-sun-solaris2.*, many 64-bit gfortran.dg tests (starting with
  gfortran.dg/allocate_class_3.f90) FAIL at -O1 and above

I'd like to make sure those are not related to your patch first.

    Rainer

Reply via email to