This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: [PATCH] Fix PR66168: ICE due to incorrect invariant register info


> From: Jeff Law [mailto:law@redhat.com]
> Sent: Wednesday, May 27, 2015 11:24 PM
> Ah, OK.  I was looking at the code prior to the call for
> can_move_invariant_reg in move_invariant_reg which implies that DEST
> can
> be a subreg, but REG can not.
> 
> But with that check in can_move_invariant_reg obviously won't matter.
> It feels like we've likely got some dead code here, but that can be a
> follow-up if you want to pursue.

Are you referring to the subreg code? It's used at the end of the function:

inv->reg = reg;
inv->orig_regno = regno;

> 
> OK for the trunk.

Thanks, committed.

Best regards,

Thomas



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]