diff options
| author | bde <bde@FreeBSD.org> | 2003-07-29 11:44:31 +0000 |
|---|---|---|
| committer | bde <bde@FreeBSD.org> | 2003-07-29 11:44:31 +0000 |
| commit | d77fd6ae884418b07affe6ebd99af9bfe1e71a2f (patch) | |
| tree | f6a7e4db07361e3fd5211d22c0f59823da18cff1 /sys | |
| parent | ebd6a7dd85ee7f269f70290638540c54be6386ff (diff) | |
| download | FreeBSD-src-d77fd6ae884418b07affe6ebd99af9bfe1e71a2f.zip FreeBSD-src-d77fd6ae884418b07affe6ebd99af9bfe1e71a2f.tar.gz | |
Don't hide the name of tmpstk, since there is no need to do so and the
HIDENAME() macro seems to be unimplementable in C. (HIDENAME() used
to use invalid token pasting using ## for the STDC case until gcc
started rejecting that; now it uses unportable token pasting using
juxtaposition in all cases.) This reduces use of HIDENAME() in the
kernel to only i386 and amd64 profiling code so that it doesn't bite
most kernels whenever gcc becomes stricter. Problems with HIDENAME()
in userland are smaller because userland mostly doesn't use strict
flags yet. There are some advantages to hiding the name of mcount,
but newer arches shouldn't do it; only amd64 does.
MFC after: 3 days
On second thoughts hide tmpstk better by staticizing it.
Diffstat (limited to 'sys')
| -rw-r--r-- | sys/i386/i386/locore.s | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/sys/i386/i386/locore.s b/sys/i386/i386/locore.s index 37d2e1f..9baee54 100644 --- a/sys/i386/i386/locore.s +++ b/sys/i386/i386/locore.s @@ -98,9 +98,8 @@ .data ALIGN_DATA /* just to be sure */ - .globl HIDENAME(tmpstk) .space 0x2000 /* space for tmpstk - temporary stack */ -HIDENAME(tmpstk): +tmpstk: .globl bootinfo bootinfo: .space BOOTINFO_SIZE /* bootinfo that we can handle */ @@ -244,7 +243,7 @@ NON_GPROF_ENTRY(btext) * the old stack, but it need not be, since recover_bootinfo actually * returns via the old frame. */ - movl $R(HIDENAME(tmpstk)),%esp + movl $R(tmpstk),%esp #ifdef PC98 /* pc98_machine_type & M_EPSON_PC98 */ |
