[quake3-bugzilla] [Bug 4352] New: Makefile: allow custom optimization flags

[bugzilla-daemon at icculus.org]

http://bugzilla.icculus.org/show_bug.cgi?id=4352

           Summary: Makefile: allow custom optimization flags
           Product: ioquake3
           Version: SVN HEAD
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P3
         Component: Misc
        AssignedTo: zakk at icculus.org
        ReportedBy: andrey.vihrov at gmail.com
         QAContact: quake3-bugzilla at icculus.org


Created an attachment (id=2207)
 --> (http://bugzilla.icculus.org/attachment.cgi?id=2207)
Proposed changes

Currently, ioquake3 Makefile respects user supplied CFLAGS, but just adds them
to the list of default ones. If the user wants to use own custom optimization
flags which conflict with default flags (OPTIMIZE and OPTIMIZEVM variables),
the user should be able to override the defaults in favor of custom CFLAGS. An
example for such use is the linux-i386 platform, where default OPTIMIZE
contains -march=i586, which is suboptimal on modern CPUs - most people building
ioquake3 for this platform will want to use -march=native, -march=core2 or
another.

Attaching a patch that makes such functionality available. The OPTIMIZE and
OPTIMIZEVM variables are not simply emptied, but rather filtered because some
platforms add -DNDEBUG to the variables' values, which should be preserved even
in case of custom CFLAGS. Another solution for this would be introducing an
additional OPTIMIZE_CPPFLAGS variable.

-- 
Configure bugmail: http://bugzilla.icculus.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.