Page MenuHomeFreeBSD

rb_tree: optimize tree rotation
ClosedPublic

Authored by dougm on Jun 19 2022, 2:59 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jul 14, 5:58 AM
Unknown Object (File)
Sun, Jul 12, 8:06 PM
Unknown Object (File)
Fri, Jul 10, 7:29 AM
Unknown Object (File)
Thu, Jul 2, 2:44 PM
Unknown Object (File)
Sun, Jun 28, 6:21 PM
Unknown Object (File)
Thu, Jun 25, 10:28 PM
Unknown Object (File)
Thu, Jun 25, 10:07 AM
Unknown Object (File)
Mon, Jun 22, 5:02 PM
Subscribers

Details

Summary

The RB_ROTATE macros begin with fetching a field via a pointer. In almost most cases, that value is one that is already in a register, and the compiler cannot infer it. So, to eliminate those needless fetches, have the caller of the RB_ROTATE macros present the data in the third macro parameter, rather than having the macro fetch it.

And a few minor style cleanups.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

dougm requested review of this revision.Jun 19 2022, 2:59 AM
dougm created this revision.

Rename variables in REMOVE_COLOR (elm->child, sib->elm) to make it look more like the INSERT_COLOR code.

Find a better (smaller) way to set 'child' in RB_INSERT_COLOR. Reduce lip3 size from 8830 to 8734.

This revision is now accepted and ready to land.Jun 25 2022, 7:30 AM
This revision was automatically updated to reflect the committed changes.