On Thu, Jan 05, 2017 at 03:20:26PM -0500, David Malcolm wrote:
> + /* Handle "reuse_rtx". */
> + if (strcmp (code_name, "reuse_rtx") == 0)
> + {
> + read_name (&name);
> + long idx = atoi (name.string);
> + /* Look it up by ID. */
> + gcc_assert (idx < m_reuse_rtx_by_id.length ());
> + return_rtx = m_reuse_rtx_by_id[idx];
> + return return_rtx;
> + }
This broke bootstrap on i686-linux (and other ILP32 hosts), because
vec.h length () returns unsigned.
Is the following ok for trunk if it passes bootstrap/regtest?
2017-01-06 Jakub Jelinek <[email protected]>
* read-rtl.c (rtx_reader::read_rtx_code): Avoid -Wsign-compare
warning.
--- gcc/read-rtl.c.jj 2017-01-06 16:58:43.000000000 +0100
+++ gcc/read-rtl.c 2017-01-06 17:22:32.105744812 +0100
@@ -1255,7 +1255,7 @@ rtx_reader::read_rtx_code (const char *c
if (strcmp (code_name, "reuse_rtx") == 0)
{
read_name (&name);
- long idx = atoi (name.string);
+ unsigned idx = atoi (name.string);
/* Look it up by ID. */
gcc_assert (idx < m_reuse_rtx_by_id.length ());
return_rtx = m_reuse_rtx_by_id[idx];
Jakub
