Replace simm32
extractor with i32_from_iconst
#8842
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This commit removes the
simm32
extractor from lowerings as it's not as useful as it was when it was first introduced. Nowadays anImm64
needs to be interpreted with the type known as well to understand whether bits being masked off is significant or not. The oldsimm32
extractor only tookImm64
meaning that it was unable to do this and wouldn't match negative numbers. This is because the high 32 bits ofImm64
were always zero andsimm64
would take thei64
value fromImm64
and try to convert it to ani32
.This commit replaces
simm32
, and uses of it, with a new extractori32_from_iconst
. This matches the preexistingi64_from_iconst
and is able to take the type of the value into account and produce a correctly sign-extended value.cc #8706