> You removed conditition with LRA_SUBREG for non-paradoxical subreg > generated for matched operands. I think that is important condition and > the comment says why. There are some 32-bit insns constraints requiring > different modes (int and fp ones) for matching operands in FP regs. The > condition prevents LRA cycling as such subreg can look invalid (in > simplify_subreg_regno) but it is used internally in LRA for matching > constraint expression and should be not reloaded. > > With this change the patch is ok for me. >
Thank you very much for the review! Patch fixed according to your comments and committed as r203169. Regards, Wei Mi.
