Re: Non-urgent issue with fs/isofs/util.c

Andreas Schwab (schwab@issan.informatik.uni-dortmund.de)
18 Sep 1998 11:52:33 +0200


"Helge Hafting" <helge.hafting@daldata.no> writes:

|> In <19980918151457.C9740@harvestroad.com.au>, on 09/18/98
|> at 03:14 PM, Ian McKellar <imckellar@harvestroad.com.au> said:
|> >I don't think we need to be to concerned right now, but changing:
|> > if (((year+2) % 4) == 0 && month > 2)
|> >to:
|> > if (((year+2) % 4) == 0 && month > 2 && year != 130) should
|> >fix it.
|> The equivalent code
|> if (!((year+2) & 3) && month > 2 && year != 130) is probably
|> faster. The somewhat expensive mod operator is
|> replaced by a bitwise "and", that is faster on most platforms.

GCC does that conversion by itself.

-- 
Andreas Schwab                                      "And now for something
schwab@issan.informatik.uni-dortmund.de              completely different"
schwab@gnu.org

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/