Skip to content

Update known failures running s2wasm results in wasm-shell#1057

Merged
sbc100 merged 1 commit intomasterfrom
update_expectations
Jun 20, 2017
Merged

Update known failures running s2wasm results in wasm-shell#1057
sbc100 merged 1 commit intomasterfrom
update_expectations

Conversation

@sbc100
Copy link
Copy Markdown
Member

@sbc100 sbc100 commented Jun 20, 2017

The two new failures are calling aboirt(). They back the
last LKGR build: https://wasm-stat.us/builders/linux/builds/20068
So its hard to say when they started failing.

The two new failures are calling aboirt().  They back the
last LKGR build: https://wasm-stat.us/builders/linux/builds/20068
So its hard to say when they started failing.

It would be nice to attribute the new successes to
certain bugfixes but I have not done that work yet.
@sbc100 sbc100 requested a review from jgravelle-google June 20, 2017 04:00
@sbc100 sbc100 merged commit 722e6a1 into master Jun 20, 2017
@sbc100 sbc100 deleted the update_expectations branch June 20, 2017 18:17
@kripken
Copy link
Copy Markdown
Member

kripken commented Jun 21, 2017

It looks like this broke the s2wasm tests. I saw them broken locally, bisected to this commit. Also, it looks like they broke on CI here, it says "2 of 4 checks passed"? @sbc100

@sbc100
Copy link
Copy Markdown
Member Author

sbc100 commented Jun 21, 2017

Strange. Sorry I didn't realize this file was actually used outside of the wasm waterfall. Will revert.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants