On 10/30/19 3:58 AM, Claudiu Zissulescu wrote: > Hi Jeff, > >> So I'm going to have to trust you on this one. It looks like you did >> more than just reorder the alternatives. For example, the constraints >> for operand0 look significantly different to me. THey're slightly >> different for operand1 as well (LR rather than Lc). > > When we moved the ARC backend to LRA, we retired the register classes covered > by `c` and `w` register constraints letters. > Now, all of them are pointing to the same register class like `r`. > So, whenever I have the opportunity, I also refactor the patterns not to use > any longer the obsolete register constraints. Probably, this is the reason > why the patch looks like it is more than reordering, but at the core it is > just that :) Thanks for the explanation. I'll keep in mind for other ARC patches as well.
jeff
