Paths

Table of Contentst

Define 'lr' as x30 on aarch64
ClosedPublic
Actions

Authored by kan on Apr 7 2017, 3:36 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Oct 2, 3:33 PM

	Unknown Object (File)
	Wed, Oct 1, 5:45 PM

	Unknown Object (File)
	Wed, Oct 1, 5:01 PM

	Unknown Object (File)
	Mon, Sep 22, 9:12 PM

	Unknown Object (File)
	Sun, Sep 21, 2:06 PM

	Unknown Object (File)
	Sep 6 2025, 7:20 PM

	Unknown Object (File)
	Aug 22 2025, 5:39 PM

	Unknown Object (File)
	Aug 8 2025, 4:41 AM

Subscribers

Details

Reviewers

andrew
emaste
imp

Group Reviewers

arm64

Commits

rS316626: Define 'lr' as x30 on aarch64

Summary

GNU toolchain does not recognize LR as standard register alias,
but clang does. Use of #define will work on both. Place the
definition into central machine/asm.h instead of patching every
affected file, as requested by plaftorm maintainers.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kan created this revision.Apr 7 2017, 3:36 PM

Herald added subscribers: emaste, andrew, imp. · View Herald TranscriptApr 7 2017, 3:36 PM

kan added reviewers: andrew, emaste, arm64.Apr 7 2017, 3:36 PM

lgtm

This revision is now accepted and ready to land.Apr 7 2017, 3:38 PM

Harbormaster failed remote builds in B8584: Diff 27182!Apr 7 2017, 3:40 PM

andrew accepted this revision.Apr 7 2017, 3:45 PM

No objection from me. I am a bit puzzled by us having a mix of lr and x30 in our .S sources though.

Closed by commit rS316626: Define 'lr' as x30 on aarch64 (authored by kan). · Explain WhyApr 7 2017, 10:58 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

arm64/

include/

asm.h

3 lines

Diff 27207

View Options

head/sys/arm64/include/asm.h

Define 'lr' as x30 on aarch64ClosedPublicActions