On Thu, Oct 11, 2012 at 11:44 PM, Teresa Johnson wrote: > + mode = targetm.machine_mode_from_attr_mode(insn);
Nit: space between "..._mode" and "(".
A test case would also be Nice To Have.
Looks OK to me otherwise, but I can't approve it.
Ciao!
Steven
