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.

Sign up for GitHub

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

Jump to bottom

Optimize reflection #823

Merged
merged 30 commits into from
Sep 10, 2024
Merged

Optimize reflection #823

merged 30 commits into from
Sep 10, 2024

Conversation

swatanabe
Copy link
Collaborator

No description provided.

@swatanabe
Replace reflect<T>::for_each with separate lists of members + helper …
0d4a542 
…functions
@swatanabe
Replace string_view get with psio::get_data_member
57a223b
@swatanabe
Remove unused overload of get
bd9af5a
@swatanabe
Remove get_by_name
fe6358d
@swatanabe
Remove Name parameter from proxy
690e357
@swatanabe
Removed out-dated comments
2e46f39
@swatanabe
Optimize lookup of reflect
31b9076
@swatanabe
Separately compile some functions
f8b8c2a
@swatanabe
Fix view<const T> for fracpack wrapped types.
d31661d
@swatanabe
Move more code out-of-line
e0aa42d
@swatanabe
Precalculating the size when packing doubles the number of instantiat…
ffaee23 
…ions
@swatanabe
Using PSIO_REFLECT inside the class body reduces the time spent in ov…
02ce20c 
…erload resolution.
@swatanabe
Replace some expensive std components
8fe8532
@swatanabe
to_key takes a const parameter
d7bec10
@swatanabe
Remove struct_tuple_type from reflect
91d0a4b
@swatanabe
Replace std::forward
374ba20
@swatanabe
Fix dispatch for views and const actions
531668f
@swatanabe
Separately compile json_token_stream
0737084
@swatanabe
The block timestamp is the start of the block, not the end. We can re…
d8c09d8 
…duce the wait by one second.
@swatanabe
We don't want the sccache daemon to stop until the build is finished.
e2b372c
@swatanabe
The tests can use a minimal install
425e42f
@swatanabe
CICD doesn't need to build debug wasms
b30e337
@swatanabe
Try to get sccache stats in the same docker run.
d514c68
@swatanabe
There's no point to using std::forward here since the actual call pas…
24131f9 
…ses the function as an lvalue
@swatanabe
Events.hpp doesn't need to include EventsTables.hpp
327e564
@swatanabe
Use separate function objects in tuple pack
bc880ba
@swatanabe
Use fold expr to implement tuple_foreach
6428859
@swatanabe
Reduce extra instantiations of std::tuple
c55f89f
@swatanabe
Completely replace tuple_remove_view
683f809
@swatanabe
Fix build
18eb397
@swatanabe swatanabe marked this pull request as ready for review September 10, 2024 13:41
@swatanabe swatanabe merged commit 32093cf into main Sep 10, 2024
4 checks passed
@swatanabe swatanabe deleted the reflect-opt branch September 10, 2024 19:26
@James-Mart James-Mart added the Refactoring Internal clean-up tasks, or eliminating tech debt label Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@James-Mart James-Mart James-Mart approved these changes

Assignees
No one assigned
Labels
Refactoring Internal clean-up tasks, or eliminating tech debt
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@swatanabe @James-Mart