amd64: Remove PMAP_INLINE
ClosedPublic
Actions

Authored by markj on Nov 2 2023, 4:38 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Aug 10, 8:20 AM

	Unknown Object (File)
	Sun, Aug 9, 10:06 PM

	Unknown Object (File)
	Sun, Aug 9, 10:02 PM

	Unknown Object (File)
	Sat, Aug 8, 3:27 AM

	Unknown Object (File)
	Sun, Jul 26, 11:42 AM

	Unknown Object (File)
	Fri, Jul 24, 6:25 PM

	Unknown Object (File)
	Wed, Jul 22, 6:23 PM

	Unknown Object (File)
	Wed, Jul 22, 3:21 PM

View All 95 Files

Subscribers

imp

Details

Reviewers

alc
kib

Commits

rG3c3a703ee657: amd64: Remove PMAP_INLINE
rG2b084923824e: amd64: Remove PMAP_INLINE

Summary

With clang it expands to "inline", which in practice may inline
externally visible functions, so the hint is likely not necessary.

No functional change intended. pmap.o is identical before and after
this patch.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Nov 2 2023, 4:38 PM

Herald added a subscriber: imp. · View Herald TranscriptNov 2 2023, 4:38 PM

markj requested review of this revision.Nov 2 2023, 4:38 PM

Harbormaster completed remote builds in B54253: Diff 129649.Nov 2 2023, 4:38 PM

alc accepted this revision.Nov 2 2023, 4:41 PM

This revision is now accepted and ready to land.Nov 2 2023, 4:41 PM

Closed by commit rG2b084923824e: amd64: Remove PMAP_INLINE (authored by markj). · Explain WhyNov 2 2023, 6:47 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG2b084923824e: amd64: Remove PMAP_INLINE.

markj added a commit: rG3c3a703ee657: amd64: Remove PMAP_INLINE.Nov 9 2023, 2:40 PM

Revision Contents
Changeset List

Path

Size

sys/

amd64/

pmap.c

18 lines

Diff 129658

View Options

amd64: Remove PMAP_INLINEClosedPublicActions