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

gcc: remove --without-multilib #16751

Merged
merged 1 commit into from
Aug 13, 2017
Merged

gcc: remove --without-multilib #16751

merged 1 commit into from
Aug 13, 2017

Conversation

fxcoudert
Copy link
Member

@fxcoudert fxcoudert commented Aug 13, 2017

9% of GCC installs use the --without-multilib option, which skips the build of 32-bit runtimes libraries. These take up 37 MB of disk space, i.e. 13% of a GCC install (287 MB). However, these users don't get bottles, and the GCC build literally takes hours.

After googling, it appears the reason for that is we used to display the following caveat about multilib:

GCC has been built with multilib support. Notably, OpenMP may not work: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60670 If you need OpenMP support you may want to brew reinstall gcc --without-multilib

I removed this caveat sometimes ago, because that bug does not actually manifest itself on Darwin. Thus there is no incompatibility between OpenMP and multilib support. Yet people still use the option for that reason, and miss out on our wonderful bottles.

In light of this, I propose that we remove the --without-multilib option.


install events in the last 30 days for gcc
================================================================================
 1 | gcc                                                      | 31,105 |  90.28%
 2 | gcc --without-multilib                                   |  3,096 |   8.99%
 3 | gcc --HEAD                                               |    188 |   0.55%
 4 | gcc --with-jit --with-nls                                |     28 |   0.08%
 5 | gcc --with-nls                                           |     10 |   0.03%

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.

3 participants