Stream: git-wasmtime

Topic: wasmtime / PR #2478 aarch64 isel: collect_address_addends...


view this post on Zulip Wasmtime GitHub notifications bot (Dec 04 2020 at 17:41):

julian-seward1 opened PR #2478 from fix-collect-address-addends to main:

…TW(negative immediate)`.

The current code doesn't correctly handle the case where ExtendOp::UXTW has
as source, a constant-producing insn that produces a negative (32-bit) value.
Then the value is incorrectly sign-extended to 64 bits (in fact, this has
already been done by ctx.get_constant(insn)), whereas it needs to be zero
extended. The obvious fix, done here, is just to force bits 63:32 of the
extension to zero, hence zero-extending it.

<!--

Please ensure that the following steps are all taken care of before submitting
the PR.

Please ensure all communication adheres to the code of conduct.
-->

view this post on Zulip Wasmtime GitHub notifications bot (Dec 04 2020 at 17:42):

julian-seward1 requested cfallin for a review on PR #2478.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 04 2020 at 17:47):

cfallin submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 04 2020 at 18:21):

julian-seward1 merged PR #2478.


Last updated: Dec 23 2024 at 12:05 UTC