Re: [PATCH][combine][obvious] Use std::swap instead of manually swapping

2015-04-27 Thread Jeff Law
On 04/27/2015 08:37 AM, Segher Boessenkool wrote: On Mon, Apr 27, 2015 at 10:55:17AM +0100, Kyrill Tkachov wrote: Precedents suggest these changes are considered obvious. So I'll commit this in a couple of days unless someone objects. Yes it's obvious. One tiny thing... @@ -9062,7 +9061,7 @

RE: [PATCH][combine][obvious] Use std::swap instead of manually swapping

2015-04-27 Thread Kyrill Tkachov
On 27/04/15 15:37, Segher Boessenkool wrote: > On Mon, Apr 27, 2015 at 10:55:17AM +0100, Kyrill Tkachov wrote: >> Precedents suggest these changes are considered obvious. >> So I'll commit this in a couple of days unless someone objects. > > Yes it's obvious. One tiny thing... > >> @@ -9062,7 +906

Re: [PATCH][combine][obvious] Use std::swap instead of manually swapping

2015-04-27 Thread Segher Boessenkool
On Mon, Apr 27, 2015 at 10:55:17AM +0100, Kyrill Tkachov wrote: > Precedents suggest these changes are considered obvious. > So I'll commit this in a couple of days unless someone objects. Yes it's obvious. One tiny thing... > @@ -9062,7 +9061,7 @@ known_cond (rtx x, enum rtx_code cond, rtx reg,