Page MenuHomeFreeBSD

sys/sys: Check for __ASSEMBLER__ in elf_common.h
ClosedPublic

Authored by andrew on Oct 16 2023, 3:07 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 18, 5:33 PM
Unknown Object (File)
Mon, Nov 18, 1:21 PM
Unknown Object (File)
Mon, Nov 18, 6:13 AM
Unknown Object (File)
Mon, Nov 11, 9:36 AM
Unknown Object (File)
Mon, Nov 11, 4:01 AM
Unknown Object (File)
Oct 21 2024, 5:16 AM
Unknown Object (File)
Oct 21 2024, 4:43 AM
Unknown Object (File)
Sep 17 2024, 3:09 AM
Subscribers

Details

Summary

When preprocessing assembly files with clang or gcc the ASSEMBLER
macro is defined. Check for this as an alternative to LOCORE in
elf_common.h so it can be included by .S files.

Sponsored by: Arm Ltd

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This is fine...

But maybe we don't need the LOCORE hack at all if all assemblers define ASSEMBLER now?

This revision is now accepted and ready to land.Oct 16 2023, 3:09 PM

I don't think we need LOCORE. I can follow up with a commit to remove LOCORE, although I'm unsure if we have anything that depends on it (either base or ports)