This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: [PATCH] expr.c:emit_push_insn


 > 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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]