Re: [PATCH][ARM][cleanup] Use IN_RANGE more often

2015-04-20 Thread Richard Earnshaw
On 20/04/15 12:03, Kyrill Tkachov wrote: > > On 18/04/15 15:18, Richard Earnshaw wrote: >> On 15/04/15 16:22, Kyrill Tkachov wrote: >>> Hi all, >>> >>> This patch goes through the arm backend and replaces expressions of the >>> form >>> a >= lo && a <= hi with IN_RANGE (a, lo, hi) which is that ti

Re: [PATCH][ARM][cleanup] Use IN_RANGE more often

2015-04-20 Thread Jakub Jelinek
On Mon, Apr 20, 2015 at 12:03:11PM +0100, Kyrill Tkachov wrote: > The definition and comment on IN_RANGE in system.h is: > /* A macro to determine whether a VALUE lies inclusively within a >certain range without evaluating the VALUE more than once. This >macro won't warn if the VALUE is un

Re: [PATCH][ARM][cleanup] Use IN_RANGE more often

2015-04-20 Thread Kyrill Tkachov
On 18/04/15 15:18, Richard Earnshaw wrote: On 15/04/15 16:22, Kyrill Tkachov wrote: Hi all, This patch goes through the arm backend and replaces expressions of the form a >= lo && a <= hi with IN_RANGE (a, lo, hi) which is that tiny bit smaller and easier to read in my opinion. I guess there's

Re: [PATCH][ARM][cleanup] Use IN_RANGE more often

2015-04-18 Thread Richard Earnshaw
On 15/04/15 16:22, Kyrill Tkachov wrote: > Hi all, > > This patch goes through the arm backend and replaces expressions of the > form > a >= lo && a <= hi with IN_RANGE (a, lo, hi) which is that tiny bit smaller > and easier to read in my opinion. I guess there's also a chance it might > make > th