Skip to content

Enable more validation tests#315

Merged
oli-obk merged 2 commits intorust-lang:masterfrom
RalfJung:validation
Aug 28, 2017
Merged

Enable more validation tests#315
oli-obk merged 2 commits intorust-lang:masterfrom
RalfJung:validation

Conversation

@RalfJung
Copy link
Member

rust-lang/rust#43457 got fixed :)

@oli-obk
Copy link
Contributor

oli-obk commented Aug 28, 2017

I have no idea why some of the tests are still failing... It works locally, but doesn't work on travis. Nevermind, these tests just don't work with full mir + validation

@oli-obk
Copy link
Contributor

oli-obk commented Aug 28, 2017

Let's turn validation back off for these tests.

@RalfJung
Copy link
Member Author

RalfJung commented Aug 28, 2017

Thanks, that looks right. I have an idea for how to fix #296; once I implemented that, I can reevaluate these tests.

@RalfJung
Copy link
Member Author

The one thing I do not understand at all is undefined_byte_read.rs. That one still works on master (the CRON job just accepted it), but it failed here?

@RalfJung
Copy link
Member Author

Turns out that was just a fluke on my end. I removed that commit.

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.

2 participants