Skip to content

change TaggedLen to store on_heap on MSB #569

Description

@alejandro-vaz

currently, our tagged length stores on the least significant bit the marker for whether the allocation is on the heap or on the stack

changing it to the MSB will allow to make incrementing/decrementing by any value simple additions/subtractions which would improve performance

however, throughout the codebase we don't make use of direct addition/subtraction to TaggedLen, and instead we rely on replacing the instance and keeping a counter on the fly, which is not nice and has unnecessary overhead. #570 aims to change this

this issue only addresses changing to MSB

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    good first issueGood for a first contributorhelp wantedNo clear candidate to implement itp-highhigh priorityr-refactorrelated to refactoring

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions