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.

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

[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
Merged

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

merged 4 commits into from
Sep 22, 2023

Commits on Sep 17, 2023

  1. [Coroutine][DebugInfo] Pre-commit test for a DISubprogram with declar… 

    …ation. (NFC)

Pre-commit test for D157184.

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

(cherry picked from commit 88a83c9)
    @DianQK
    DianQK committed Sep 17, 2023
    Configuration menu
    Copy the full SHA
    6ee1391 View commit details
    Browse the repository at this point in the history

  2. [Coroutine][DebugInfo] Update the linkage name of the declaration of … 

    …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)
    @DianQK
    DianQK committed Sep 17, 2023
    Configuration menu
    Copy the full SHA
    737daf7 View commit details
    Browse the repository at this point in the history

  3. [Coroutine][DebugInfo] Reduced test case for `coro-async-declaration.… 

    …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)
    @DianQK
    DianQK committed Sep 17, 2023
    Configuration menu
    Copy the full SHA
    1b345e9 View commit details
    Browse the repository at this point in the history

  4. [Coroutine][DebugInfo] Remove the memory attributes on coro-async-dec… 

    …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 committed Sep 17, 2023
    Configuration menu
    Copy the full SHA
    51ee87d View commit details
    Browse the repository at this point in the history