Skip to content

Conversation

@alex-rass-88
Copy link
Contributor

@alex-rass-88 alex-rass-88 commented Sep 23, 2024

This enable sanitizer flags for C lang.

What does this PR do?

Enable gcc sanitizer and visibility flags for C language.

How does this PR change Premake's behavior?

Adds sanitize and visibility to C

Anything else we should know?

closes #2033

Did you check all the boxes?

  • Focus on a single fix or feature; remove any unrelated formatting or code changes
  • Add unit tests showing fix or feature works; all tests pass
  • Mention any related issues (put closes #XXXX in comment to auto-close issue when PR is merged)
  • Follow our coding conventions
  • Minimize the number of commits
  • Align documentation to your changes

You can now support Premake on our OpenCollective. Your contributions help us spend more time responding to requests like these!

@Jarod42
Copy link
Contributor

Jarod42 commented Sep 23, 2024

[Off Topic]: it is x to check the checkbox, not *

- [x] checked
- [ ] unchecked
  • checked
  • unchecked

@samsinsane
Copy link
Member

Moving these flags broke some unit tests, you'll need to update them.

@samsinsane samsinsane merged commit 5174a98 into premake:master Sep 24, 2024
@Jarod42
Copy link
Contributor

Jarod42 commented Nov 23, 2024

@alex-rass-88 : Thanks to look at #2335

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.

Support of sanitize { "Address" } to C-language

3 participants