On Nov 25, 2013, at 1:01 AM, Eric Botcazou <ebotca...@adacore.com> wrote: >> Richi has asked the we break the wide-int patch so that the individual port >> and front end maintainers can review their parts without have to go through >> the entire patch. This patch covers the sparc port. > > OK if you change the type of 'low' in solaris_pragma_align to unsigned HWI.
Ok, here is what I committed (to the wide-int branch), thanks. Committed revision 205365. Index: gcc/ChangeLog.wide-int =================================================================== --- gcc/ChangeLog.wide-int (revision 205364) +++ gcc/ChangeLog.wide-int (working copy) @@ -92,7 +92,7 @@ * config/rs6000/rs6000.md Use const_scalar_int_operand. Handle CONST_WIDE_INT. * config/sol2-c.c - (solaris_pragma_align): Use tree_fits_uhwi_p. + (solaris_pragma_align): Change low to unsigned HWI. Use tree_fits_uhwi_p. * config/sparc/sparc.c: Include wide-int.h. (sparc_fold_builtin): Use wide-int interfaces. * config/vax/vax.c: Include wide-int.h. Index: gcc/config/sol2-c.c =================================================================== --- gcc/config/sol2-c.c (revision 205364) +++ gcc/config/sol2-c.c (working copy) @@ -86,7 +86,7 @@ solaris_pragma_align (cpp_reader *pfile { tree t, x; enum cpp_ttype ttype; - HOST_WIDE_INT low; + unsigned HOST_WIDE_INT low; if (pragma_lex (&x) != CPP_NUMBER || pragma_lex (&t) != CPP_OPEN_PAREN)