Other than that, it looks good to go to me.
Oops, my x86 bootstrap with this patch just died in PRE:
insn-recog.c:17561: internal compiler error: virtual array
basic_block_info[5349]: element 3537031808 out of bounds in
insert_occ_in_preorder_dt_order_1, at tree-ssa-pre.c:903
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.
make[1]: *** [insn-recog.o] Error 1
make[1]: Leaving directory `/build/tree-ssa/2003-10-21/gcc'
make: *** [stage2_build] Error 2
My source tree is a couple of days old, I'll try it with a fresh tree
too.