Skip to content

Debug Info: Insert dbg.value intrinsics immediately after the instruction they describe. #4940

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 23, 2016
Merged

Debug Info: Insert dbg.value intrinsics immediately after the instruction they describe. #4940

merged 2 commits into from
Sep 23, 2016

Conversation

adrian-prantl
Copy link
Contributor 

Debug Info: Insert dbg.value intrinsics immediately after the instruction
they describe.

This improves the live ranges of local variables during debugging.

<rdar://problem/26627376>

@adrian-prantl
Copy link
Contributor Author

@swift-ci please test and merge

@adrian-prantl
Copy link
Contributor Author

macOS error is unrelated.

@adrian-prantl
Copy link
Contributor Author

@swift-ci please test OS X platform

@swift-ci
Copy link
Contributor

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - fc7dbe2d8c28de13ef69a0c41d9ca73cf20dbd0d
Test requested by - @adrian-prantl

@adrian-prantl
Debug Info: Insert dbg.value intrinsics immediately after the instruc…
fa28b5e 
…tion

they describe.

This improves the live ranges of local variables during debugging.

<rdar://problem/26627376>
@adrian-prantl
Copy link
Contributor Author

This time it found an actual error.

@adrian-prantl
Copy link
Contributor Author

@swift-ci please test and merge

1 similar comment
@adrian-prantl
Copy link
Contributor Author

@swift-ci please test and merge

@swift-ci swift-ci merged commit 7d7eb08 into swiftlang:master Sep 23, 2016
@e78l
Copy link

e78l commented Sep 25, 2016

Hello,
The "Debug Info..." commit seems to have caused build failures in swiftpm for me on Linux. Based on https://bugs.swift.org/browse/SR-2754, I would guess it makes LLVM use a lot of memory when compiling https://github.com/apple/swift-package-manager/blob/master/Sources/Utility/StringMangling.swift

swiftpm built fine after reverting this locally (git revert -m 1 7d7eb08)

MaxDesiatov pushed a commit that referenced this pull request Sep 7, 2023
@kateinoigakukun
Merge pull request #4940 from swiftwasm/katei/merge-main-2022-09-24
67affbd 
Merge main 2022-09-24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@adrian-prantl @swift-ci @e78l