Messages in this thread | | | Date | Wed, 13 Apr 2011 00:16:58 +0300 | From | Alexey Dobriyan <> | Subject | Re: [PATCH] remove abs64() |
| |
On Tue, Apr 12, 2011 at 02:10:40PM -0700, Andrew Morton wrote: > On Tue, 12 Apr 2011 14:07:26 -0700 > Randy Dunlap <rdunlap@xenotime.net> wrote: > > > > + __builtin_choose_expr( \ > > > + __builtin_types_compatible_p(typeof(_x), long long), \ > > > + (unsigned long long)({ _x < 0 ? -_x : _x; }), \ > > > + _x))))); \ > > > +}) > > > > that is better? > > I think so. > > It's a bit concerning that it changes the return type of abs().
I haven't read every abs() user, but, yes, sizeof(abs()) silently changing is the issue.
| |