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

Fix issue with imports not being removed #189

Open
larryk85 opened this issue May 30, 2023 · 0 comments
Open

Fix issue with imports not being removed #189

larryk85 opened this issue May 30, 2023 · 0 comments
Labels
bug Something isn't working triage

Comments

@larryk85
Copy link
Contributor

CDT will compile smart contracts and include unused imports into the final smart contract.

To fix this the place to make changes will be in cdt-lld and revolve around how we determine liveness of those imports.

We are doing something wrong, so we will need to do a better analysis of liveness of those imports and prune those that are not used in the smart contract and those that are not reachable from 'apply' (the main entry point).

@larryk85 larryk85 added the bug Something isn't working label May 30, 2023
@larryk85 larryk85 added this to the CDT 4.0.0 milestone May 30, 2023
@ericpassmore ericpassmore modified the milestones: CDT 4.0.0, CDT 5.0.0-rc1 Aug 11, 2023
@ericpassmore ericpassmore modified the milestones: CDT 4.1.0-rc1, Future Milestone Aug 24, 2023
@arhag arhag removed this from the Future Milestone milestone Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage
Projects
Status: Icebox
Development

No branches or pull requests

6 participants