On 04/29/2015 01:55 AM, Mikhail Maltsev wrote:
I probably would have done separate patches for the std::swap
changes. They're not really related to the rtx subclasses work.
OK, sending 2 separate patches. Note that they a not "commutative":
std::swap should be applied before the main one, because one of the
swaps in do_compare_rtx_and_jump uses a single temporary variable of
type rtx for swapping labels and for storing generic rtl expressions
(this could be worked around, of course, but I think that would be just
a waste of time).
I've applied the swap patch. I want to look over the subclass patch a
final time before applying it.
If you're going to continue this work, you should probably get
write-after-approval access so that you can commit your own approved
changes.
jeff
