-
Notifications
You must be signed in to change notification settings - Fork 31
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
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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).
The text was updated successfully, but these errors were encountered: