[PATCH] reorganize simplify_const_relational_operation

Dave Korn dave.korn@artimi.com
Wed Sep 19 10:51:00 GMT 2007


On 18 September 2007 21:17, Eric Botcazou wrote:

>> Okay, so I'll s/enum cr_flags/int/ (except in the definition of course).
> 
> Thanks.  Btw, what do you think about the extension code just above?

> 	  l0u &= ((HOST_WIDE_INT) 1 << width) - 1;
> 	  l1u &= ((HOST_WIDE_INT) 1 << width) - 1;
> 
> 	  if (l0s & ((HOST_WIDE_INT) 1 << (width - 1)))
> 	    l0s |= ((HOST_WIDE_INT) (-1) << width);
> 
> 	  if (l1s & ((HOST_WIDE_INT) 1 << (width - 1)))
> 	    l1s |= ((HOST_WIDE_INT) (-1) << width);

  I think that's a really awful choice of variable names in a monospaced font!
:-)


    cheers,
      DaveK
-- 
Can't think of a witty .sigline today....



More information about the Gcc-patches mailing list