On 12/18/2009 03:40 AM, Laurent Desnogues wrote:
I'm not sure I really like that rewrite, I find it hard to read.
Convince me it's better :-)
Would it be more obvious if I used a table instead of a switch?
The code size reduction is
Num: Value Size Type Bind Vis Ndx Name
before:
40: 000039a0 793 FUNC LOCAL DEFAULT 1 tcg_out_brcond2
after:
40: 00003a50 412 FUNC LOCAL DEFAULT 1 tcg_out_brcond2
table-ized:
40: 00003a50 224 FUNC LOCAL DEFAULT 1 tcg_out_brcond2
But I'm not married to the patch; let's drop it if there's a question of
it holding up the rest of the series.
r~