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

[Coroutine][DebugInfo] Update the linkage name of the declaration of coro-split functions in the debug info. #7485

Merged
merged 4 commits into from
Sep 22, 2023

Conversation

DianQK
Copy link

@DianQK DianQK commented Sep 17, 2023

Cherry pick to stable/20230725 as discussed in #7168.

(cherry picked from commit 88a83c9)
(cherry picked from commit ca1a5b3)
(cherry picked from commit 3aed002)
(cherry picked from commit 19b664d)

@adrian-prantl @felipepiovezan @drodriguez

…ation. (NFC)

Pre-commit test for D157184.

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

(cherry picked from commit 88a83c9)
…coro-split functions in the debug info.

This patch adds the linkage name update to DISubprogram's declaration after 6ce76ff.

Reviewed By: ChuanqiXu

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

(cherry picked from commit ca1a5b3)
…ll`. (NFC)

Based on the description at https://reviews.llvm.org/D157177#inline-1529005, I try to improve the test case.

Reviewed By: fdeazeve

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

(cherry picked from commit 3aed002)
…laration.ll (NFC) (llvm#66088)

According to @drodriguez's reminder in
swiftlang#7168 (comment),
`memory` breaks the backport to the apple branch.

And this is irrelevant to that test. Delete to get better a test case.

(cherry picked from commit 19b664d)
@DianQK DianQK changed the title Stable/20230725 [Coroutine][DebugInfo] Update the linkage name of the declaration of coro-split functions in the debug info. Sep 17, 2023
@felipepiovezan
Copy link

@swift-ci please test llvm

@felipepiovezan felipepiovezan merged commit 0db3f71 into swiftlang:stable/20230725 Sep 22, 2023
1 of 2 checks passed
@DianQK DianQK deleted the stable/20230725 branch September 22, 2023 21:08
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.

None yet

2 participants