Re: Fix for PR sanitize/65000 introduces an unusual gcc_assert

2015-10-28 Thread Richard Henderson
On 10/28/2015 12:09 PM, Richard Biener wrote: On October 28, 2015 7:07:54 PM GMT+01:00, Richard Henderson wrote: On 10/28/2015 10:47 AM, Alexander Monakov wrote: Hello Richard, Your commit to fix PR 65000 (pasted below) introduced gcc_assert (ri = (int)ri); I'm unclear what is meant the

Re: Fix for PR sanitize/65000 introduces an unusual gcc_assert

2015-10-28 Thread Richard Biener
On October 28, 2015 7:07:54 PM GMT+01:00, Richard Henderson wrote: >On 10/28/2015 10:47 AM, Alexander Monakov wrote: >> Hello Richard, >> >> Your commit to fix PR 65000 (pasted below) introduced >>gcc_assert (ri = (int)ri); >> >> I'm unclear what is meant there; if equality test was meant, th

Re: Fix for PR sanitize/65000 introduces an unusual gcc_assert

2015-10-28 Thread Richard Henderson
On 10/28/2015 10:47 AM, Alexander Monakov wrote: Hello Richard, Your commit to fix PR 65000 (pasted below) introduced gcc_assert (ri = (int)ri); I'm unclear what is meant there; if equality test was meant, that looks suspicious to me because truncating conversion is implementation-defined.

Fix for PR sanitize/65000 introduces an unusual gcc_assert

2015-10-28 Thread Alexander Monakov
Hello Richard, Your commit to fix PR 65000 (pasted below) introduced gcc_assert (ri = (int)ri); I'm unclear what is meant there; if equality test was meant, that looks suspicious to me because truncating conversion is implementation-defined. Can you please comment? (I found this after noticing