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