Skip to content
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

test padding in format strings #626

Merged
merged 4 commits into from
Feb 13, 2019
Merged

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented Feb 9, 2019

Waiting for rust-lang/rust#58324 to land

@RalfJung RalfJung added the S-blocked-on-rust Status: Blocked on landing a Rust PR label Feb 9, 2019
pietroalbini added a commit to pietroalbini/rust that referenced this pull request Feb 10, 2019
miri: give non-generic functions a stable address

This makes Miri correctly handle format string parameters despite rust-lang#58320.

Matching Miri PR: rust-lang/miri#626

r? @oli-obk
Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request Feb 10, 2019
miri: give non-generic functions a stable address

This makes Miri correctly handle format string parameters despite rust-lang#58320.

Matching Miri PR: rust-lang/miri#626

r? @oli-obk
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Feb 10, 2019
miri: give non-generic functions a stable address

This makes Miri correctly handle format string parameters despite rust-lang#58320.

Matching Miri PR: rust-lang/miri#626

r? @oli-obk
@RalfJung RalfJung added S-waiting-on-nightly and removed S-blocked-on-rust Status: Blocked on landing a Rust PR labels Feb 11, 2019
@RalfJung
Copy link
Member Author

Somehow the "is_generic" stuff doesn't work... even for a function like

fn f<T: Default>() -> T {
    Default::default()
}

when turning it to a function pointer, I am seeing

TRACE 2019-02-13T09:01:32Z: rustc::mir::interpret: create_fn_alloc(Instance { def: Item(DefId(0/0:3 ~ function_pointers[317d]::f[0])), substs: [] }): is_generic=false

How can f have empty substs?!?

@RalfJung
Copy link
Member Author

Oh never mind... I edited the wrong checkout of Miri... ^^

@RalfJung RalfJung merged commit 60a39df into rust-lang:master Feb 13, 2019
@RalfJung RalfJung deleted the format-padding branch March 26, 2019 09:15
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.

1 participant