Stream: git-wasmtime

Topic: wasmtime / PR #5797 x64: Improve codegen for vectors with...


view this post on Zulip Wasmtime GitHub notifications bot (Feb 16 2023 at 00:55):

alexcrichton opened PR #5797 from improve-x64-constant-vec-shifts to main:

I stumbled across this working on #5795 and figured this was a nice opportunity to improve the codegen here.

<!--

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 (Feb 16 2023 at 00:57):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 16 2023 at 00:57):

alexcrichton created PR review comment:

I'll admit I don't know what's happening here. This I believe is the constant pool area and capstone is trying to decode it as instructions, but I don't know why after this change, which should generate a smaller constant pool, is generating more instructions here. I would hazard a guess that capstone automatically stoped beforehand for some reason and now it's keeping-on-going, but I'm not sure.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 16 2023 at 01:42):

jameysharp submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 16 2023 at 01:42):

jameysharp created PR review comment:

I think this is at least partially the same issue that @elliottt is tackling in #5798. We've discussed ways to entirely quit trying to disassemble constants since we think the machbuffer knows where they are, but one step at a time.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 16 2023 at 15:33):

alexcrichton updated PR #5797 from improve-x64-constant-vec-shifts to main.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 16 2023 at 18:30):

jameysharp submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 16 2023 at 18:30):

jameysharp created PR review comment:

Since #5798 is merged, can you try rebasing and see if the filetests look any better now?

view this post on Zulip Wasmtime GitHub notifications bot (Feb 16 2023 at 18:33):

jameysharp submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 16 2023 at 18:33):

jameysharp created PR review comment:

Nevermind, I see you already did and it looks like it didn't help. I'm going to declare that this is harmless and move on with reviewing the rest of this PR.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 16 2023 at 18:38):

jameysharp submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 16 2023 at 19:57):

alexcrichton updated PR #5797 from improve-x64-constant-vec-shifts to main.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 16 2023 at 19:57):

alexcrichton has enabled auto merge for PR #5797.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 16 2023 at 21:19):

alexcrichton merged PR #5797.


Last updated: Dec 23 2024 at 12:05 UTC