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

Remove unused global __Copyright #26

Conversation

ragusaa
Copy link
Contributor

@ragusaa ragusaa commented Jan 31, 2023

Unused globals are causing warnings in the runtime due to the list indices getting out of sync in the list of Globals and ConstantExprs. This PR stores the copyright statement as metadata, so there is no unused global inserted into the runtime.

Unused globals are causing warnings in the runtime due to the list
indices getting out of sync in the list of Globals and ConstantExprs.
This PR stores the copyright statement as metadata, so there is no
unused global inserted into the runtime.
@ragusaa ragusaa force-pushed the CLAM-2195-RemoveUnusedCopyrightVariable branch from a1e2a51 to 0c8827d Compare January 31, 2023 21:34
@micahsnyder
Copy link
Contributor

Hey @ragusaa Just realized this PR is really stale. Was it waiting for my review?

@ragusaa
Copy link
Contributor Author

ragusaa commented Aug 15, 2023

I think it was, I must have forgotten to ask about it (obviously). Let's just close it and get it done with the clang 16 upgrade.

@micahsnyder
Copy link
Contributor

@ragusaa the PR that CLAM-2195 references in fixing this says it was fixed by #23

I am confused.

@ragusaa ragusaa closed this Aug 31, 2023
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.

2 participants