This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: small emit_case_nodes tweek
On Jun 14, 2001, Alexandre Oliva <aoliva@redhat.com> wrote:
> On Jun 14, 2001, Jan Hubicka <jh@suse.cz> wrote:
>> + /* Instead of doing two branches emit test (index-low) <= (high-low). */
> [snip]
>> + emit_cmp_and_jump_insns (new_index,
>> + expand_expr (new_bound, NULL_RTX,
>> + VOIDmode, 0),
>> + GT, NULL_RTX, mode, 1, 0,
> Shouldn't this be GTU? Think of low == INT_MIN and high == INT_MAX,
> for example.
Never mind. I've just realized the `1' in the same line requests an
unsigned cmp.
--
Alexandre Oliva Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer aoliva@{cygnus.com, redhat.com}
CS PhD student at IC-Unicamp oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist *Please* write to mailing lists, not to me