Skip to content

Conversation

cam900
Copy link

@cam900 cam900 commented May 11, 2022

This PR will be remove column limits, mentioned into #3572.
Also, I add useful functions for Bit array and vectors, for future expansion and usage.

cam900 added 3 commits May 8, 2022 03:07
Add operator overrides and more functions for bit vector and arrays
This commit fixes remain boundary issues.
@ocornut
Copy link
Owner

ocornut commented May 23, 2022

This creates 4 extra heap allocations per table which is not acceptable unfortunately (both in term of allocations count and in terms of extra indirection and strain on cache lines). Those buffers would need to be included in the single table->RawData buffer while keeping usage terse and obvious.

I also don't think overriding operator& is judicious and desirable here.

@Konrni Konrni mentioned this pull request Aug 2, 2022
ocornut added a commit that referenced this pull request Jan 20, 2023
… code-paths are already touching the cold parts.

Only exception being TableSetColumnIndex() with same column number but that's an odd case.
Will break PR #6094 #3572 #5305 #4876 but those not need to be necessarily updated: we got enough reference to finish that feature.
ocornut added a commit that referenced this pull request Jan 20, 2023
@ocornut
Copy link
Owner

ocornut commented Jan 20, 2023

Now pushed an official change for that: lifting limits and performances are equivalent.
See details on this comment : #6094 (comment)

@ocornut ocornut closed this Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants