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

Backport D99579 #100

Merged
merged 1 commit into from
Apr 17, 2021
Merged

Conversation

nagisa
Copy link
Member

@nagisa nagisa commented Apr 16, 2021

Addresses majority of the situations in which the issue described in rust-lang/rust#83139 occurs.

While probing stack, the stack register is moved without dwarf
information, which could cause panic if unwind the backtrace.
This commit only add annotation for the inline stack probe case.
Dwarf information for the loop case should be done in another
patch and need further discussion.

Reviewed By: nagisa

Differential Revision: https://reviews.llvm.org/D99579
@nagisa
Copy link
Member Author

nagisa commented Apr 16, 2021

r? @cuviper / @nikic

@cuviper
Copy link
Member

cuviper commented Apr 17, 2021

LGTM!

@cuviper cuviper merged commit ea6bb26 into rust-lang:rustc/12.0-2021-04-15 Apr 17, 2021
nikic pushed a commit that referenced this pull request Jul 10, 2021
While probing stack, the stack register is moved without dwarf
information, which could cause panic if unwind the backtrace.
This commit only add annotation for the inline stack probe case.
Dwarf information for the loop case should be done in another
patch and need further discussion.

Reviewed By: nagisa

Differential Revision: https://reviews.llvm.org/D99579

Co-authored-by: YangKeao <keao.yang@yahoo.com>
vext01 added a commit to vext01/llvm-project that referenced this pull request Nov 15, 2023
100: Get ykllvm to provide enough info to identify a zero-length call. r=ltratt a=vext01



Co-authored-by: Edd Barrett <vext01@gmail.com>
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