Differential D20845

Remove a goto from vm_map_wire_locked
ClosedPublic
Actions

Authored by dougm on Jul 3 2019, 9:19 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, May 21, 12:45 AM

	Unknown Object (File)
	Wed, May 13, 5:17 PM

	Unknown Object (File)
	Thu, May 7, 2:04 PM

	Unknown Object (File)
	Thu, May 7, 1:31 PM

	Unknown Object (File)
	Wed, May 6, 10:19 PM

	Unknown Object (File)
	Mon, May 4, 9:20 AM

	Unknown Object (File)
	Fri, May 1, 11:19 AM

	Unknown Object (File)
	Wed, Apr 29, 9:48 PM

View All 96 Files

Subscribers

imp

Details

Reviewers

alc

Commits

rS349713: Eliminate a goto and a label in vm_map_wire_locked by inserting an 'else'.

Summary

Removing the goto and its target label, and inserting an 'else' in front of the next 'if' to achieve the same control flow. produced no change in the number of machine instructions or their order on an amd64.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dougm created this revision.Jul 3 2019, 9:19 PM

alc accepted this revision.Jul 3 2019, 10:35 PM

This revision is now accepted and ready to land.Jul 3 2019, 10:35 PM

Closed by commit rS349713: Eliminate a goto and a label in vm_map_wire_locked by inserting an 'else'. (authored by dougm). · Explain WhyJul 3 2019, 10:42 PM

This revision was automatically updated to reflect the committed changes.

dougm added a commit: rS349713: Eliminate a goto and a label in vm_map_wire_locked by inserting an 'else'..

Herald added a subscriber: imp. · View Herald TranscriptJul 3 2019, 10:42 PM

Revision Contents
Changeset List

Path

Size

head/

sys/

vm/

vm_map.c

5 lines

Diff 59375

View Options

Remove a goto from vm_map_wire_lockedClosedPublicActions