Project

General

Profile

Actions

Bug #1355

closed

-O3 deemed uncessary

Added by roberto (Ollivier Robert) over 15 years ago. Updated over 13 years ago.

Status:
Closed
Assignee:
-
Target version:
-
ruby -v:
ruby 1.9.2dev (2009-03-31) [i386-darwin9.6.0]
Backport:
[ruby-core:23130]

Description

=begin
I do not see the real point in forcing all compilation with -O3 and it makes overriding the optimization level overly complicated. Let people choose the level they want.
=end


Files

o3.diff (275 Bytes) o3.diff Patch to remove -O3 by default. roberto (Ollivier Robert), 04/06/2009 05:29 PM
Actions #1

Updated by shyouhei (Shyouhei Urabe) over 15 years ago

=begin
The real point? gcc -O2 was a crap. We've recieved many "bug" reports that ruby cannot compile.

If you want that silly -O2 enabled, type ./configure optflags=-O2
=end

Actions #2

Updated by roberto (Ollivier Robert) over 15 years ago

=begin
There are not that many differences between -O2 and -O3 (mainly -finline-functions IIRC) so I'm a bit surprised there. Anyway, I'd remove any "default" level to let people choose except for -O. The main problem is that if I do "CFLAGS=-O2 ./configure", it is end up with "-O -O3" (which is really -O3). Either add a "--cflags" parameter to "configure.in" or remove -O3 altogether. Your way of specifying optflags is correct of course but violates POLA in my book. Best regards.
=end

Actions #3

Updated by nobu (Nobuyoshi Nakada) over 15 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

=begin
Applied in changeset r23142.
=end

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0