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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |aoliva at gcc dot gnu.org,
                   |                            |jakub at gcc dot gnu.org,
                   |                            |rguenth at gcc dot gnu.org

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
The r228175
        (promote_ssa_mode): Disregard BLKmode from promote_decl, and            
        bypass TYPE_MODE to get the actual vector mode.                         

+  /* Bypass TYPE_MODE when it maps vector modes to BLKmode.  */                
+  if (mode == BLKmode)                                                         
+    {                                                                          
+      gcc_assert (VECTOR_TYPE_P (type));                                       
+      mode = type->type_common.mode;                                           
+    }                                                                          
+                                                                               

change looks highly suspicious, that introduces in the IL vector modes that
aren't really supported.  If that was done because some SSA_NAMEs might have
underlying decl that has DECL_MODE, I'd say we should either treat all
DECL_MODE similarly to TYPE_MODE, or add more hacks where for vector type decls
we just ignore DECL_MODE and use corresponding TYPE_MODE (I think we have quite
a few of those already).

Reply via email to