amd64: macroify copyinstr
AbandonedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Jan 4, 9:40 AM

	Unknown Object (File)
	Nov 2 2024, 1:43 PM

	Unknown Object (File)
	Oct 25 2024, 1:29 AM

	Unknown Object (File)
	Sep 17 2024, 5:22 PM

	Unknown Object (File)
	Sep 13 2024, 7:23 AM

	Unknown Object (File)
	Sep 4 2024, 8:13 AM

	Unknown Object (File)
	Aug 23 2024, 1:59 AM

	Unknown Object (File)
	Aug 8 2024, 5:15 PM

View All 20 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