Skip to content
This repository was archived by the owner on Feb 5, 2019. It is now read-only.

Backport a fix for duplicate debug data errors - #93

Merged
alexcrichton merged 1 commit into
rust-lang:rust-llvm-release-4-0-1from
dotdash:dbg_update
Oct 10, 2017
Merged

alexcrichton merged 1 commit into
rust-lang:rust-llvm-release-4-0-1from
dotdash:dbg_update

Conversation

@dotdash

@dotdash dotdash commented Oct 10, 2017

Copy link
Copy Markdown

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

Fixes problems seen in rust-lang/rust#44573

Some passes might duplicate calls to llvm.dbg.declare creating
duplicate frame index expression which currently trigger an assertion
which is meant to catch erroneous, overlapping fragment declarations.
But identical frame index expressions are just redundant and don't
actually conflict with each other, so we can be more lenient and just
ignore the duplicates.

Reviewers: aprantl, rnk

Subscribers: llvm-commits, JDevlieghere

Differential Revision: https://reviews.llvm.org/D38540
@alexcrichton
alexcrichton merged commit 76fce77 into rust-lang:rust-llvm-release-4-0-1 Oct 10, 2017
@alexcrichton

Copy link
Copy Markdown
Member

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants