diff options
| author | Max Howell | 2012-02-16 17:23:11 +0000 |
|---|---|---|
| committer | Max Howell | 2012-02-16 18:19:00 +0000 |
| commit | be953bbb85c457abb3928c1ac05f939e78ac3bb6 (patch) | |
| tree | ef1e45c11607a03510411cd4324ca0c038b185a8 | |
| parent | ec66d1f956f229f822a93ab45853ce1b1ad68aa4 (diff) | |
| download | brew-be953bbb85c457abb3928c1ac05f939e78ac3bb6.tar.bz2 | |
Use -Os rather than -O3
This was probably a stupid decision in the first place. Who wants lengthy compile times for debatable gain? Apple use Os in all Xcode projects, so I'm guessing its the best choice.
Also I have long suspected the Image Magick performance issues Homebrew is supposedly susceptible to were because of O3.
| -rw-r--r-- | Library/Homebrew/extend/ENV.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Library/Homebrew/extend/ENV.rb b/Library/Homebrew/extend/ENV.rb index 609264cd1..0820c55e4 100644 --- a/Library/Homebrew/extend/ENV.rb +++ b/Library/Homebrew/extend/ENV.rb @@ -19,8 +19,8 @@ module HomebrewEnvExtension self['CMAKE_PREFIX_PATH'] = "#{HOMEBREW_PREFIX}" end - # llvm allows -O4 however it often fails to link and is very slow - self['CFLAGS'] = self['CXXFLAGS'] = "-O3 #{SAFE_CFLAGS_FLAGS}" + # Os is the default Apple uses for all its stuff so let's trust them + self['CFLAGS'] = self['CXXFLAGS'] = "-Os #{SAFE_CFLAGS_FLAGS}" self.send self.compiler # we must have a working compiler! |