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

[SR-14137] [C++] Cache failed imports. #56517

Open
zoecarver opened this issue Feb 1, 2021 · 2 comments
Open

[SR-14137] [C++] Cache failed imports. #56517

zoecarver opened this issue Feb 1, 2021 · 2 comments
Labels
c++ interop Feature: Interoperability with C++ compiler The Swift compiler in itself improvement

Comments

@zoecarver
Copy link
Collaborator

Previous ID SR-14137
Radar rdar://problem/83405988
Original Reporter @zoecarver
Type Improvement
Status In Progress
Resolution
Additional Detail from JIRA
Votes 0
Component/s Compiler
Labels Improvement, CxxInterop, StarterBug
Assignee thomassw66 (JIRA)
Priority Medium

md5: e6fd025c695731ad4c62c7d1b91945d7

Issue Description:

Currently, we cache decls after we successfully import them. However, if we fail to import a decl, we should (but currently do not) cache them too. I suspect that this will be a big performance improvement for the compiler because currently, we can't import complex templates or dependent types, which are very expensive to traverse.

This will require fixing a few visitors along the way.

@zoecarver
Copy link
Collaborator Author

@swift-ci create

@swift-ci swift-ci transferred this issue from apple/swift-issues Apr 25, 2022
@zoecarver zoecarver added c++ interop Feature: Interoperability with C++ and removed CxxInterop labels Apr 25, 2022
@Nallu-Swami
Copy link

Has this been assigned ? i am willing to work on this issue

@egorzhdan egorzhdan removed the good first issue Good for newcomers label May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ interop Feature: Interoperability with C++ compiler The Swift compiler in itself improvement
Projects
None yet
Development

No branches or pull requests

3 participants