[Z80asm-devel] [patch #3396] Makefile: remove -Werror, change to -O2, make clean

[Bas Wijnen]

This mail is an automated notification from the patch tracker
 of the project: Z80 assembler.

/**************************************************************************/
[patch #3396] Latest Modifications:

Changes by: 
                Bas Wijnen <address@hidden>
'Date: 
                Wed 09/29/2004 at 12:46 (Europe/Amsterdam)

            What     | Removed                   | Added
---------------------------------------------------------------------------
          Resolution | None                      | Applied
         Assigned to | None                      | shevek
              Status | Open                      | Closed







/**************************************************************************/
[patch #3396] Full Item Snapshot:

URL: <http://savannah.nongnu.org/patch/?func=detailitem&item_id=3396>
Project: Z80 assembler
Submitted by: H. Peter Anvin
On: Tue 09/28/2004 at 20:16

Category:  None
Priority:  5 - Normal
Resolution:  Applied
Privacy:  Public
Assigned to:  shevek
Originator Email:  
Status:  Closed


Summary:  Makefile: remove -Werror, change to -O2, make clean

Original Submission:  For gcc, -O3 or higher means "inline everything", which 
for code such as compilers/assemblers which contain lots of function calls 
often cause worse performance due to cache effects, and more importantly, adds 
a lot to the code size (35% on Linux/x86-64, 47% on Linux/i386) for no 
measurable gain -- especially given that it might cause code to be evicted from 
the L1 I$.

-Werror breaks compiling on some broken platforms, e.g. Cygwin, which otherwise 
would compile correctly but with warnings.

Finally, fix "make clean".

Follow-up Comments
------------------


-------------------------------------------------------
Date: Tue 09/28/2004 at 23:00       By: H. Peter Anvin <hpanvin>
Even more Cygwin friendly (unfortunately I have to use Cygwin because of stupid 
hardware synthesis tools which are $0 on 'doze and $3000 on Linux.)







File Attachments
-------------------

-------------------------------------------------------
Date: Tue 09/28/2004 at 23:00  Name: Makefile.diff  Size: 1.09KB   By: hpanvin
Updated patch - more Cygwin friendly
http://savannah.nongnu.org/patch/download.php?item_id=3396&amp;item_file_id=3711

-------------------------------------------------------
Date: Tue 09/28/2004 at 20:16  Name: Makefile.diff  Size: 1.04KB   By: hpanvin

http://savannah.nongnu.org/patch/download.php?item_id=3396&amp;item_file_id=3710






For detailed info, follow this link:
<http://savannah.nongnu.org/patch/?func=detailitem&item_id=3396>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/