-
Notifications
You must be signed in to change notification settings - Fork 12.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
error output between self-hosted versus cross-compile target differs significantly for some tests #23926
Comments
… 64-bit targets. See also rust-lang#23926.
Ah this case
Maybe the easiest thing would be to make 32- and 64-bit variants of that same test. (That or conditionalize the size we feed into the array construction based on pointer size.) Update: another thing to maybe check is to see if trying to allocate an |
Triage: no changes I'm aware of. |
There are two other tests currently tagged with this issue, const-eval-overflow-3.rs and const-eval-overflow-3b.rs, both in compile-fail. The issue with both appears to be the same as with the previous test mentioned; we either can or cannot create and evaluate the const array depending on platform. |
I wonder if we might be able to leverage the revision system in some way here. That, or as I noted earlier: make pointer-size-specific variants of each test. (And then presumably ignore the tests on platforms that cannot support the 64-bit pointer-sized variant.) |
Better solution: make target-specific versions of each |
AFAICT, we do not have the same const-eval issues that we used to when rust-lang#23926 was filed. (Probably because of the switch to miri for const-evaluation.)
…r=nikomatsakis Update tests to account for cross-platform testing and miri. Fix rust-lang#23926
…r=nikomatsakis Update tests to account for cross-platform testing and miri. Fix rust-lang#23926
…r=nikomatsakis Update tests to account for cross-platform testing and miri. Fix rust-lang#23926
…r=nikomatsakis Update tests to account for cross-platform testing and miri. Fix rust-lang#23926
…r=nikomatsakis Update tests to account for cross-platform testing and miri. Fix rust-lang#23926
The error output between self-hosted versus cross-compile target can differ significantly for some tests.
In some cases one can get around this via
error-pattern
, but it does not work universally.This bug is to mark such cases, and hopefully go back and identify why the error output is so different (in some cases the reason may be self-evident, but that was not universally true).
The text was updated successfully, but these errors were encountered: