Skip to content

Conversation

aheejin
Copy link
Member

@aheejin aheejin commented Dec 14, 2023

_VECTOR or _ARRAY defines in wasm-delegations-fields.def are supposed to be defined in terms of their non-vector/array counterparts when undefined. This removes empty _VECTOR/_ARRAY defines when including wasm-delegations-fields.def, while adding definitions for DELEGATE_GET_FIELD in case it is missing.

`_VECTOR` or `_ARRAY` defines in `wasm-delegations-fields.def` are supposed to be defined in terms of their non-vector/array counterparts when undefined. This removes empty `_VECTOR`/`_ARRAY` defines when including `wasm-delegations-fields.def`, while adding definitions for `DELEGATE_GET_FIELD` in case it is missing.
Copy link
Member

@kripken kripken left a comment

Choose a reason for hiding this comment

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

Good find!

@aheejin aheejin merged commit cad983c into WebAssembly:main Dec 14, 2023
@aheejin aheejin deleted the no_vector_array_def branch December 14, 2023 18:35
radekdoulik pushed a commit to dotnet/binaryen that referenced this pull request Jul 12, 2024
`_VECTOR` or `_ARRAY` defines in `wasm-delegations-fields.def` are supposed to be defined in terms of their non-vector/array counterparts when undefined. This removes empty `_VECTOR`/`_ARRAY` defines when including `wasm-delegations-fields.def`, while adding definitions for `DELEGATE_GET_FIELD` in case it is missing.
@gkdn gkdn mentioned this pull request Aug 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants