Skip to content

Conversation

tmandry
Copy link
Member

@tmandry tmandry commented Feb 12, 2020

Fix for rust-lang/rust#68794. Tested across a few different platforms in rust-lang/rust#69045, and it looks like it hasn't broken anything.

cc @Mark-Simulacrum @Amanieu

@Mark-Simulacrum
Copy link
Member

Once this lands into master and is confirmed by downstream users as both not breaking anything and fixing the bug, we will likely want to consider a beta backport as well. (Though the regression is stable/stable at this point, so it is less important than for a stable/beta regression, in some sense).

@alexcrichton alexcrichton merged commit cae3e6e into rust-lang:master Feb 12, 2020
@tmandry tmandry deleted the probestack-hidden branch February 12, 2020 06:29
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Feb 12, 2020
…n-DPC

Update compiler-builtins to 0.1.25

- rust-lang/compiler-builtins#340 Set probestack visibility to hidden on ELF targets

Fixes rust-lang#68794.

Tested in CI: rust-lang#69045.

r? @Mark-Simulacrum
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