Re: -mregparm=3 (was Re: [PATCH] i386 do_machine_check() is redundant.

Valdis.Kletnieks
2003-09-30T14:37:46Z

[-- Attachment #1: Type: text/plain, Size: 587 bytes --]

On Tue, 30 Sep 2003 00:55:13 EDT, Robert Love said:

> Hm, we may need to do something like:
>
> #define abs(n) __builtin_abs((n))
>
> because -ffreestanding implies -fno-builtin, which disables use of
> built-in functions that do not begin with __builtin.

Well, abs() is the only one I tripped over in my config. I'm sure there's others
lurking elsewhere in the kernel tree.

The bigger question is whether a patch to support -ffreestanding would be a
good idea - with proper use of the __builtin_* stuff it *should* work, and it will
hopefully cause better kernel code hygiene..

[-- Attachment #2: Type: application/pgp-signature, Size: 226 bytes --]