Wasmtime GitHub notifications bot (Jan 29 2021 at 09:32): Wasmtime GitHub notifications bot (Jan 29 2021 at 09:32):kaseyc opened PR #2622 from operand to main:
This is in preparation for refactoring all x64::Inst arms to use OperandSize.
Current uses of OperandSize fall into two categories:
1. XMM operations which require 32/64 bit operands
2. Immediates which only care about 64-bit or not.Adds assertions to existing Inst constructors to check that they are passed valid sizes.
This change also removes the implicit widening of 1 and 2 byte values to 4 bytes. from_bytes() is only used by category 2, so removing this behavior will not change any visible behavior.Overall this change should be a no-op.
Wasmtime GitHub notifications bot (Jan 31 2021 at 02:41):kaseyc updated PR #2622 from operand to main:
This is in preparation for refactoring all x64::Inst arms to use OperandSize.
Current uses of OperandSize fall into two categories:
1. XMM operations which require 32/64 bit operands
2. Immediates which only care about 64-bit or not.Adds assertions to existing Inst constructors to check that they are passed valid sizes.
This change also removes the implicit widening of 1 and 2 byte values to 4 bytes. from_bytes() is only used by category 2, so removing this behavior will not change any visible behavior.Overall this change should be a no-op.
Wasmtime GitHub notifications bot (Jan 31 2021 at 08:21):kaseyc updated PR #2622 from operand to main:
This is in preparation for refactoring all x64::Inst arms to use OperandSize.
Current uses of OperandSize fall into two categories:
1. XMM operations which require 32/64 bit operands
2. Immediates which only care about 64-bit or not.Adds assertions to existing Inst constructors to check that they are passed valid sizes.
This change also removes the implicit widening of 1 and 2 byte values to 4 bytes. from_bytes() is only used by category 2, so removing this behavior will not change any visible behavior.Overall this change should be a no-op.
Wasmtime GitHub notifications bot (Feb 01 2021 at 01:41):kaseyc updated PR #2622 from operand to main:
This is in preparation for refactoring all x64::Inst arms to use OperandSize.
Current uses of OperandSize fall into two categories:
1. XMM operations which require 32/64 bit operands
2. Immediates which only care about 64-bit or not.Adds assertions to existing Inst constructors to check that they are passed valid sizes.
This change also removes the implicit widening of 1 and 2 byte values to 4 bytes. from_bytes() is only used by category 2, so removing this behavior will not change any visible behavior.Overall this change should be a no-op.
Last updated: Nov 22 2024 at 17:03 UTC