Stream: git-wasmtime

Topic: wasmtime / PR #4942 Port flag-based ops to ISLE (AArch64)


view this post on Zulip Wasmtime GitHub notifications bot (Sep 22 2022 at 10:04):

dheaton-arm opened PR #4942 from isle-flags to main:

Ported the existing implementations of the following opcodes for AArch64 to ISLE:

Also fixed the implementation of true{if,ff} for the interpreter.

Copyright (c) 2022 Arm Limited

<!--

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 (Sep 22 2022 at 10:36):

dheaton-arm updated PR #4942 from isle-flags to main.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 22 2022 at 22:44):

cfallin submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 22 2022 at 22:44):

cfallin created PR review comment:

Wow, I'm surprised the test was missing this before -- that was a clear miscompilation that I guess we never saw only because we always used the result of iadd_ifcout directly as well. In any case, this PR fixes the issue!

view this post on Zulip Wasmtime GitHub notifications bot (Sep 22 2022 at 22:44):

cfallin submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 22 2022 at 22:44):

cfallin merged PR #4942.


Last updated: Dec 23 2024 at 12:05 UTC