Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nix Waypoint::ap_coloc #248

Open
1 task
yakra opened this issue Apr 30, 2023 · 0 comments
Open
1 task

nix Waypoint::ap_coloc #248

yakra opened this issue Apr 30, 2023 · 0 comments

Comments

@yakra
Copy link
Owner

yakra commented Apr 30, 2023

ap_coloc isn't needed for every waypoint, only graph points.
It could conceivably be nixed, and stored in a std:pair with the corresponding waypoint in hi_priority_points & low_priority_points for graph generation.

Syntax might be a drag though, short of switching to C++17 & using structured bindings.

  • Do hi_priority_points & low_priority_points go out of scope, or hang around? When can these vectors be deleted or at least cleared?

sizeof would still be 104 on FreeBSD & 112 on Ubuntu. Still above the magic 96 B for fitting 2 objects in 3 cache lines.
88 B on CentOs though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant