amd64: macroify copyinstr
AbandonedPublic
Actions

Authored by mjg on Sep 22 2018, 11:48 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Apr 17, 10:46 AM

	Unknown Object (File)
	Mar 7 2024, 11:32 PM

	Unknown Object (File)
	Feb 20 2024, 1:24 AM

	Unknown Object (File)
	Jan 12 2024, 7:13 PM

	Unknown Object (File)
	Dec 29 2023, 2:56 AM

	Unknown Object (File)
	Dec 23 2023, 2:14 AM

	Unknown Object (File)
	Sep 28 2023, 7:11 AM

	Unknown Object (File)
	Jun 12 2023, 4:57 AM

View All 9 Files

Subscribers

imp

Details

Reviewers

kib

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 19752

Event Timeline

mjg created this revision.Sep 22 2018, 11:48 AM

Herald added a subscriber: imp. · View Herald TranscriptSep 22 2018, 11:48 AM

Harbormaster completed remote builds in B19752: Diff 48351.Sep 22 2018, 11:48 AM

mjg added a child revision: D17288: amd64: depessimize copyinstr.Sep 22 2018, 11:51 AM

kib added inline comments.Sep 22 2018, 4:50 PM

sys/amd64/amd64/support.S
974	Why do you need a second copy of this prologue ?

mjg marked an inline comment as done.Sep 23 2018, 2:54 PM

mjg added inline comments.

sys/amd64/amd64/support.S
974	I would have to add a special case to the macro so that one primitive gets a label this can jump to. i can do that, but I think it would be uglier. Later I want to move all fault handling to the end of the file.

went in as part of r338970

Revision Contents
Changeset List

Path

Size

sys/

amd64/

support.S

73 lines

Diff 48351

View Options

amd64: macroify copyinstrAbandonedPublicActions