patchZ80 assembler - Patches: patch #3396, Makefile: remove -Werror, change...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #3396: Makefile: remove -Werror, change to -O2, make clean

Submitter:  H. Peter Anvin <hpanvin>
Submitted:  Tue 28 Sep 2004 08:16:36 PM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  shevek Open/Closed:  Closed

Tue 28 Sep 2004 11:00:34 PM UTC, comment #1: 

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.)

H. Peter Anvin <hpanvin>
Tue 28 Sep 2004 08:16:36 PM UTC, 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".

H. Peter Anvin <hpanvin>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #5538:  Makefile.diff added by hpanvin (1KiB - application/octet-stream - Updated patch - more Cygwin friendly)
file #7905:  Makefile.diff added by hpanvin (1KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

 

Follow 5 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2004-09-29 shevek StatusNone Done
    Assigned toNone shevek
    Open/ClosedOpen Closed
2004-09-28 hpanvin Attached File- Added Makefile.diff, #3711
2004-09-28 hpanvin Attached File- Added Makefile.diff, #3710

Back to the top

Powered by Savane 3.13-4b48.
Corresponding source code