On 09/11/2012 10:53 AM, Michael Matz wrote:
Hi,
the operands cache is ugly. This patch removes it at least for the def
operands, saving three pointers for roughly each normal statement (the
pointer in gsbase, and two pointers from def_optype_d). This is
relatively easy to do, because all statements except ASMs have at most one
def (and one vdef), which themself aren't pointed to by something else,
unlike the use operands which have more structure for the SSA web.
Yeah, a bit has changed since the original implementation. There is only
ever one vdef, and not much else in the way of multiple def's ever
arrived, So I think this makes sense.
Note that when I introduce gimple_atomic (delayed until next year :-P),
there will be at least one more statement with 2 results.. the
__atomic_cmpxchg node. so the iterator code will grow a little bit more.
Andrew
