-
Notifications
You must be signed in to change notification settings - Fork 635
build-sys: enable LTO #3922
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
build-sys: enable LTO #3922
Conversation
Signed-off-by: Masatake YAMATO <[email protected]>
Signed-off-by: Masatake YAMATO <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3922 +/- ##
=======================================
Coverage 85.36% 85.36%
=======================================
Files 234 234
Lines 56123 56125 +2
=======================================
+ Hits 47907 47909 +2
Misses 8216 8216 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Masatake YAMATO <[email protected]>
Signed-off-by: Masatake YAMATO <[email protected]>
dced70c
to
08bb69f
Compare
Close universal-ctags#2885. Signed-off-by: Masatake YAMATO <[email protected]>
doesn't consider adding a configure option like --enable-lto ? We could enable LTO by default in release mode. We also allow user to disable it. |
Why not enable LTO when cross compiling? |
Suggested by @leleliu008. Signed-off-by: Masatake YAMATO <[email protected]>
@leleliu008 Thank you. I added --enable-lto/--disable-lto.
I don't know whether my change works well with cross-compiling.
is enough?
this may be too simple to make my change work well with cross-compiling. |
LTO is nothing to do with whether cross compiling or not. I guess it would work if you just removing the cross compiling checking condition, I am on a journey, and didn't bring my laptop, I can't confirm it right now. |
O.K. I will merge this pull request that disables LTO when cross-compiling. |
Close #2885.