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)
Sat, Jan 18, 11:02 PM
Unknown Object (File)
Sat, Jan 11, 2:35 AM
Unknown Object (File)
Mon, Jan 6, 2:38 PM
Unknown Object (File)
Nov 28 2024, 2:40 PM
Unknown Object (File)
Nov 23 2024, 10:34 AM
Unknown Object (File)
Nov 22 2024, 12:53 AM
Unknown Object (File)
Nov 18 2024, 5:33 PM
Unknown Object (File)
Nov 18 2024, 1:21 PM
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)