[PATCH] Fix PR middle-end/33088

Eric Botcazou ebotcazou@libertysurf.fr
Wed Dec 12 22:43:00 GMT 2007


> I think the idea and implementation are sound.  But, did you verify
> that the places
> we set DECL_GIMPLE_REG_P in the gimplifier are properly guarded with the
> changed predicate?  That is, shouldn't the occurances of
>
>       if ((TREE_CODE (TREE_TYPE (parm)) == COMPLEX_TYPE
>
>            || TREE_CODE (TREE_TYPE (parm)) == VECTOR_TYPE)
>
> ...
>         DECL_GIMPLE_REG_P (parm) = 1;
>
> changed to use the is_gimple_reg_type predicate?  (at least in the
> gimplifier)

It seems to me that I already answered to this question in the message. :-)

The rationale is that the flag is only really read through is_gimple_reg.

-- 
Eric Botcazou



More information about the Gcc-patches mailing list