small emit_case_nodes tweek

Alexandre Oliva aoliva@redhat.com
Thu Jun 14 04:24:00 GMT 2001


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.

-- 
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



More information about the Gcc-patches mailing list