Stream: git-wasmtime

Topic: wasmtime / PR #1838 Vcode refactor helper data structures...


view this post on Zulip Wasmtime GitHub notifications bot (Jun 11 2020 at 04:44):

jlb6740 edited PR #1838 from vcode-rm-xmm-r-r-x64-enum-case to master:

<!--

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 (Jun 11 2020 at 04:51):

jlb6740 updated PR #1838 from vcode-rm-xmm-r-r-x64-enum-case to master:

<!--

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 (Jun 11 2020 at 05:11):

jlb6740 updated PR #1838 from vcode-rm-xmm-r-r-x64-enum-case to master:

<!--

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 (Jun 17 2020 at 07:03):

jlb6740 updated PR #1838 from vcode-rm-xmm-r-r-x64-enum-case to master:

<!--

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 (Jun 18 2020 at 07:02):

jlb6740 updated PR #1838 from vcode-rm-xmm-r-r-x64-enum-case to master:

<!--

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 (Jun 22 2020 at 18:46):

jlb6740 updated PR #1838 from vcode-rm-xmm-r-r-x64-enum-case to master:

<!--

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 (Jun 22 2020 at 18:50):

jlb6740 updated PR #1838 from vcode-rm-xmm-r-r-x64-enum-case to master:

<!--

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 (Jun 23 2020 at 07:29):

jlb6740 updated PR #1838 from vcode-rm-xmm-r-r-x64-enum-case to master:

<!--

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 (Jun 24 2020 at 16:40):

bnjbvr submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 24 2020 at 16:40):

bnjbvr created PR Review Comment:

I would have assumed that the tests could be kept, but using the xmm_mov_rm_r constructor instead of the xmm_r_r, and moving these close to the other xmm_mov_rm_r tests. Can you do this here please? (Unless this is a redundant ask because similar tests existed with the other Inst)

view this post on Zulip Wasmtime GitHub notifications bot (Jun 24 2020 at 16:40):

bnjbvr created PR Review Comment:

ditto here

view this post on Zulip Wasmtime GitHub notifications bot (Jun 24 2020 at 16:40):

bnjbvr submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 24 2020 at 16:40):

bnjbvr created PR Review Comment:

Bad rebase here: I think the previous code can be reused effectively by replacing xmm_r_r with xmm_mov_rm_r and adapting to this function's API. Can you do this here please?

view this post on Zulip Wasmtime GitHub notifications bot (Jun 24 2020 at 16:40):

bnjbvr submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 24 2020 at 20:24):

jlb6740 submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 24 2020 at 20:24):

jlb6740 created PR Review Comment:

Hi .. yes. I included redundant tests earlier when creating the xmm_mov_rm_r structure: See here: https://github.com/bytecodealliance/wasmtime/blob/f2f770626511c08c632c52e5f868ac13170687be/cranelift/codegen/src/isa/x64/inst/emit_tests.rs#L2400-L2409

view this post on Zulip Wasmtime GitHub notifications bot (Jun 24 2020 at 20:25):

jlb6740 submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 24 2020 at 20:25):

jlb6740 created PR Review Comment:

See above. This test is already included I just modified the registers as suggested by @julian-seward1

view this post on Zulip Wasmtime GitHub notifications bot (Jun 24 2020 at 20:49):

jlb6740 updated PR #1838 from vcode-rm-xmm-r-r-x64-enum-case to master:

<!--

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 (Jun 24 2020 at 22:14):

jlb6740 requested bnjbvr for a review on PR #1838.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 25 2020 at 12:31):

bnjbvr submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 25 2020 at 12:31):

bnjbvr merged PR #1838.


Last updated: Dec 23 2024 at 12:05 UTC