Skip to content
This repository has been archived by the owner on Mar 20, 2024. It is now read-only.

Implement vector-of-vector comparison in move-native. #226

Merged
merged 1 commit into from Jul 9, 2023

Conversation

nvjle
Copy link

@nvjle nvjle commented Jul 9, 2023

This patch extends the baseline move-native vec_cmp_eq routine to handle vector<vector<T>>.

Uncomment test_vec_vec_bool in the stdlib-bcs serializer test (which now passes).

Add more tests to vec.move to exercise vec_cmp_eq with vec-of-vec.

This patch extends the baseline `move-native` `vec_cmp_eq` routine
to handle vector<vector<T>>.

Uncomment `test_vec_vec_bool` in the stdlib-bcs serializer test
(which now passes).

Add more tests to vec.move to exercise vec_cmp_eq with vec-of-vec.
Copy link
Collaborator

@jcivlin jcivlin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nvjle nvjle merged commit af5ebfc into anza-xyz:llvm-sys Jul 9, 2023
4 checks passed
@nvjle nvjle linked an issue Jul 9, 2023 that may be closed by this pull request
ksolana pushed a commit that referenced this pull request Jul 17, 2023
This patch extends the baseline `move-native` `vec_cmp_eq` routine
to handle vector<vector<T>>.

Uncomment `test_vec_vec_bool` in the stdlib-bcs serializer test
(which now passes).

Add more tests to vec.move to exercise vec_cmp_eq with vec-of-vec.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Implement more comparison ops for vectors of struct etc.
2 participants