On Tue, 1 May 2012, Manuel L?pez-Ib??ez wrote:

> +static tree expr_original_type (tree expr)

That should be

static tree
expr_original_type (tree expr)

with a comment explaining the semantics of the function.

> +{
> +  STRIP_NOPS (expr);

Are you sure you want STRIP_NOPS rather than STRIP_SIGN_NOPS here?  If so, 
could you ensure there are comments explaining why removing sign changes 
is safe in this context?

-- 
Joseph S. Myers
jos...@codesourcery.com

Reply via email to