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

chore: Add release and LLVM asan build options #143

Merged
merged 1 commit into from
Nov 26, 2020

Conversation

JFreegman
Copy link
Member

@JFreegman JFreegman commented Nov 19, 2020

The release now uses -O2 and flto, and has no debug symbols. -Wmissing-field-initializer was removed due to false positives on newer versions of clang.


This change is Reviewable

@JFreegman JFreegman added the chore Updating grunt tasks etc; no production code change label Nov 19, 2020
@JFreegman JFreegman added this to the 0.9.x milestone Nov 19, 2020
@JFreegman JFreegman force-pushed the debug branch 6 times, most recently from 9dab329 to d3f9837 Compare November 19, 2020 19:00
@JFreegman JFreegman changed the title chore: Add debug and LLVM asan build options chore: Add release and LLVM asan build options Nov 19, 2020
@JFreegman JFreegman force-pushed the debug branch 6 times, most recently from a649018 to ec8457f Compare November 19, 2020 20:09
@TokTok TokTok deleted a comment Nov 19, 2020
@ghost
Copy link

ghost commented Nov 19, 2020

Congratulations 🎉. DeepCode analyzed your code in 0.617 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

@JFreegman JFreegman force-pushed the debug branch 6 times, most recently from 2fc0628 to 6a19836 Compare November 26, 2020 20:51
The release build uses -O2 and flto, and has no debug symbols.
-Wmissing-field-initializer was removed due to false positives
on newer versions of clang
@JFreegman JFreegman merged commit 3f2826b into TokTok:master Nov 26, 2020
@JFreegman JFreegman deleted the debug branch November 26, 2020 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Updating grunt tasks etc; no production code change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants