[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