Stream: git-wasmtime

Topic: wasmtime / PR #2180 Simd negate x64


view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2020 at 06:44):

jlb6740 opened PR #2180 from simd-negate-x64 to main:

<!--

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 02 2020 at 06:44):

jlb6740 requested bnjbvr for a review on PR #2180.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2020 at 06:44):

jlb6740 requested cfallin and bnjbvr for a review on PR #2180.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2020 at 06:45):

jlb6740 requested cfallin, abrown and bnjbvr for a review on PR #2180.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2020 at 06:47):

jlb6740 updated PR #2180 from simd-negate-x64 to main:

<!--

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 02 2020 at 17:17):

jlb6740 updated PR #2180 from simd-negate-x64 to main:

<!--

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 07 2020 at 06:56):

jlb6740 updated PR #2180 from simd-negate-x64 to main:

<!--

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 07 2020 at 15:05):

bnjbvr submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 07 2020 at 15:05):

bnjbvr created PR Review Comment:

uber nit: can you add a \n before this case statement, please? (and after it, too)

view this post on Zulip Wasmtime GitHub notifications bot (Sep 07 2020 at 15:05):

bnjbvr submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 07 2020 at 15:05):

bnjbvr created PR Review Comment:

uber nit: can you add a dot at the end of this sentence, please?

view this post on Zulip Wasmtime GitHub notifications bot (Sep 07 2020 at 15:05):

bnjbvr created PR Review Comment:

In theory, ineg can also accept non-vector inputs (I think this is addressed in #2158 even). Thus can you remove the packed word from this panic message, please?

view this post on Zulip Wasmtime GitHub notifications bot (Sep 07 2020 at 15:05):

bnjbvr created PR Review Comment:

Can you add a comment that we can't preemptively use dst as the temporary since it might alias the src input, please?

view this post on Zulip Wasmtime GitHub notifications bot (Sep 07 2020 at 15:05):

bnjbvr created PR Review Comment:

How about naming this tmp? It is unclear to me what the relation of this temporary to lhs is, and this is really just a temporary value.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 07 2020 at 17:06):

jlb6740 submitted PR Review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 07 2020 at 17:06):

jlb6740 created PR Review Comment:

Got it. Will try to avoid this in the future.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 07 2020 at 18:20):

jlb6740 updated PR #2180 from simd-negate-x64 to main:

<!--

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 07 2020 at 18:21):

jlb6740 submitted PR Review.

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

jlb6740 created PR Review Comment:

just named it to tmp.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 07 2020 at 18:56):

jlb6740 merged PR #2180.


Last updated: Dec 23 2024 at 13:07 UTC