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

Add "used" attribute to XML data symbol when creating compressed config #346

Merged
merged 1 commit into from
Jan 16, 2024

Conversation

lukash
Copy link
Contributor

@lukash lukash commented Jan 15, 2024

The attribute((used)) tells the compiler not to make the symbol local when optimizing. This happens when LTO is used when building a VESC package.

This change seems to be the only one needed to build a functional package with LTO.

The __attribute__((used)) tells the compiler not to make the symbol
local when optimizing. This happens when LTO is used when building a
VESC package.
Copy link
Contributor

github-actions bot commented Jan 15, 2024

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@lukash
Copy link
Contributor Author

lukash commented Jan 15, 2024

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Jan 15, 2024
@vedderb vedderb merged commit feb641c into vedderb:master Jan 16, 2024
1 check passed
@github-actions github-actions bot locked and limited conversation to collaborators Jan 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants