[PATCH] expr.c:emit_push_insn

Kaveh R. Ghazi ghazi@caip.rutgers.edu
Fri May 11 13:15:00 GMT 2001


 > From: Mark Mitchell <mark@codesourcery.com>
 > 
 > Kaveh, this is the kind of project you've excelled at in the past.
 > Would you be interested in tackling this?  It's the kind of thing that
 > could make the code considerably clearer over time.

It seems like there's agreement on only defining a bit_alignment.
That's fine, but I see two issues.

1.  What heuristic should I use to find where bit_alignment should be
    *used*?  (Preferably a grep expr.)  I tried '(unsigned|int).*align'
    as a first guess and got hundreds of hits, many (most?) of which
    look legitimate.

2.  As you can guess from my regex, there isn't a consensus in the
    code as to whether the type is unsigned or int.

		--Kaveh
--
Kaveh R. Ghazi			Engagement Manager / Project Services
ghazi@caip.rutgers.edu		Qwest Internet Solutions



More information about the Gcc-patches mailing list