Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upFix failing rpass-valgrind test #29896
Comments
nrc
added
P-high
T-compiler
labels
Nov 17, 2015
This comment has been minimized.
This comment has been minimized.
|
cc @nagisa |
This comment has been minimized.
This comment has been minimized.
|
cc @rust-lang/compiler (I wish labelling T-compiler did this automatically) |
This comment has been minimized.
This comment has been minimized.
|
What's the actual errors here? Running valgrind without flags reports:
And running with more flags:
(which looks benign) |
This comment has been minimized.
This comment has been minimized.
|
Happens with simple check-stage1-rpass-valgrind
|
This comment has been minimized.
This comment has been minimized.
|
Ah I think that may have been fixed by #29897 |
nrc
self-assigned this
Nov 19, 2015
This comment has been minimized.
This comment has been minimized.
|
Confirmed - we now pass all rpass-valgrind tests. |
nrc
closed this
Nov 20, 2015
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
nrc commentedNov 17, 2015
This is blocking turning valgrind tests back on.
exit-flushes.rs is apparently the culprit.
cc @edunham, @nikomatsakis, @alexcrichton
(p-high since it is blocking fixing our test infra)