This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix inconsistent use of integer types in gcc/lto-streamer-out.c
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Andris Pavenis <andris dot pavenis at iki dot fi>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, DJ Delorie <dj at delorie dot com>
- Date: Mon, 16 Nov 2015 12:12:19 +0100
- Subject: Re: [PATCH] Fix inconsistent use of integer types in gcc/lto-streamer-out.c
- Authentication-results: sourceware.org; auth=none
- References: <5648408B dot 9040107 at iki dot fi>
On Sun, Nov 15, 2015 at 9:21 AM, Andris Pavenis <andris.pavenis@iki.fi> wrote:
> This fixes use of pointers different unsigned integer types as function
> parameter.
> Function prototype is (see gcc/tree-streamer.h):
>
> bool streamer_tree_cache_lookup (struct streamer_tree_cache_d *, tree,
> unsigned *);
>
> gcc/lto-streamer-out.c passes uint32_t int * as parameter to this method in
> 2 places.
> Current type unisgned is used elsewhere in the same file.
>
> uint32_t is not guaranteed to be the same as unsigned (for DJGPP uint32_t is
> actually
> unsigned long). That causes compile failure for DJGPP native build.
Ok.
Thanks,
Richard.
> Andris
>
> 2015-11-15 Andris Pavenis <andris.pavenis@iki.fi>
>
> * gcc/lto-streamer-out.c (write_global_references): Adjust integer type
> (lto_output_decl_state_refs): Likewise
>