weaselab/conflict-set
1
Fork 1
Code Issues 5 Pull Requests Actions 1 Packages Releases 13 Activity

Fuse fast path switch into regular switch to reduce branches #15

New Issue
Closed
opened 2024-03-21 16:52:04 +00:00 by andrew · 1 comment
andrew commented 2024-03-21 16:52:04 +00:00
Owner
Copy Link
Copy Source

conflict-set/ConflictSet.cpp Line 864 in a4d1f91670
Node *&getOrCreateChild(Node *&self, uint8_t index,

https://git.weaselab.dev/weaselab/conflict-set/src/commit/a4d1f9167048f51cfecc14bbeb5235e18574fe56/ConflictSet.cpp#L864
andrew commented 2024-03-28 17:16:53 +00:00
Author
Owner
Copy Link
Copy Source

Tried it, but it didn't seem to improve instruction count according to cachegrind

Tried it, but it didn't seem to improve instruction count according to cachegrind
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something is not working
 enhancement
New feature
 investigation
wontfix
This won't be fixed
missing-test-coverage
Issues having to do with missing test coverage
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
andrew weaselbot
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: weaselab/conflict-set#15