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

--- Comment #28 from Peter Bergner <bergner at gcc dot gnu.org> ---
(In reply to Andrew Macleod from comment #25)
> Wonder if it was suppose to be something like:
> 
> 
>    /* Vector pair and vector quad support.  */
>    if (TARGET_EXTRA_BUILTINS)
>      {
> -      tree oi_uns_type = make_unsigned_type (256);
> -      vector_pair_type_node = build_distinct_type_copy (oi_uns_type);
> +      vector_pair_type_node = make_unsigned_type (256);
>        SET_TYPE_MODE (vector_pair_type_node, POImode);
>        layout_type (vector_pair_type_node);
>        lang_hooks.types.register_builtin_type (vector_pair_type_node,
>                                               "__vector_pair");
>  
> -      tree xi_uns_type = make_unsigned_type (512);
> -      vector_quad_type_node = build_distinct_type_copy (xi_uns_type);
> +      vector_quad_type_node = make_unsigned_type (512);
>        SET_TYPE_MODE (vector_quad_type_node, PXImode);
>        layout_type (vector_quad_type_node);
>        lang_hooks.types.register_builtin_type (vector_quad_type_node,

So this passed bootstrap and regtesting with no regressions.

Is this really the correct fix?  Don't we want a distinct type compared to the
unsigned type returned from make_unsigned_type()?

Reply via email to