Stream: git-wasmtime

Topic: wasmtime / Issue #1788 x86_32: legalize br{z,nz}.i64


view this post on Zulip Wasmtime GitHub notifications bot (May 29 2020 at 17:19):

whitequark commented on Issue #1788:

cc @iximeow

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

github-actions[bot] commented on Issue #1788:

Subscribe to Label Action

cc @bnjbvr

<details>
This issue or pull request has been labeled: "cranelift", "cranelift:meta"

Thus the following users have been cc'd because of the following labels:

To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.

Learn more.
</details>

view this post on Zulip Wasmtime GitHub notifications bot (May 29 2020 at 19:07):

whitequark commented on Issue #1788:

No logs for failed builds?..

view this post on Zulip Wasmtime GitHub notifications bot (May 29 2020 at 20:34):

tschneidereit commented on Issue #1788:

No logs for failed builds?..

Yeah, something's wrong here. I just retriggered the tests in the hope that this was an intermittent infra issue.

view this post on Zulip Wasmtime GitHub notifications bot (May 30 2020 at 04:43):

whitequark commented on Issue #1788:

I just retriggered the tests in the hope that this was an intermittent infra issue.

I think that broke again, though less severely.

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

iximeow commented on Issue #1788:

I tried rerunning again (because why not) only to see the same mac builds fail. Still no logs or indication of how to acquire them..

view this post on Zulip Wasmtime GitHub notifications bot (May 30 2020 at 09:50):

tschneidereit commented on Issue #1788:

Just tried again, because indeed: why not, but no change. Given that

@whitequark would you be up for opening a fresh PR for these changes? :fingers_crossed:that makes it go away.

view this post on Zulip Wasmtime GitHub notifications bot (May 30 2020 at 09:51):

tschneidereit edited a comment on Issue #1788:

Just tried again, because indeed: why not, but no change. Given that

I wonder if something's just borked about this PR.

@whitequark would you be up for opening a fresh PR for these changes? :fingers_crossed:that makes it go away.

view this post on Zulip Wasmtime GitHub notifications bot (May 30 2020 at 10:26):

whitequark commented on Issue #1788:

@tschneidereit https://github.com/bytecodealliance/wasmtime/pull/1794

view this post on Zulip Wasmtime GitHub notifications bot (May 30 2020 at 10:58):

tschneidereit commented on Issue #1788:

@whitequark thank you! Given that #1794 worked, would it make sense to close this and open a new PR with identical patches? It seems implausible that the difference in code chances between this PR and #1794 would translate into this difference in CI success.

view this post on Zulip Wasmtime GitHub notifications bot (May 30 2020 at 11:59):

whitequark commented on Issue #1788:

Given that #1794 worked, would it make sense to close this and open a new PR with identical patches? It seems implausible that the difference in code chances between this PR and #1794 would translate into this difference in CI success.

Er, sorry, I actually made a mistake here--I pushed a wrong branch. Let me do the right thing...

view this post on Zulip Wasmtime GitHub notifications bot (May 30 2020 at 12:01):

whitequark commented on Issue #1788:

https://github.com/bytecodealliance/wasmtime/pull/1795


Last updated: Nov 22 2024 at 16:03 UTC