Opened 16 years ago

Last modified 15 years ago

#647 new Defect

Remove -fomit-frame-pointer and -ffast-math gcc flags

Reported by: mjakubicek Owned by: davea
Priority: Minor Milestone: Undetermined
Component: Client - Build Version: 6.6.37
Keywords: patch Cc: mjakubicek

Description

Is there any particular reason to have those flags set as default? They can make debugging much harder or impossible and should be avoided if possible. I ask because they are currently prohibited for Fedora, so that I had to remove them anyway in the Fedora package. (patch included)

Attachments (1)

boinc-gccflags.patch​ (1.9 KB) - added by mjakubicek 16 years ago.
Patch for the Makefile.in and Makefile.am -- remove -fomit-frame-pointer and -ffast-math

Download all attachments as: .zip

Change History (9)

Changed 16 years ago by mjakubicek

Attachment: boinc-gccflags.patch​ added

Patch for the Makefile.in and Makefile.am -- remove -fomit-frame-pointer and -ffast-math

comment:1 Changed 16 years ago by Nicolas

Component: Undetermined → Client - Build
Owner: set to romw
Priority: Undetermined → Minor

comment:2 Changed 16 years ago by Nicolas

Keywords: patch added

comment:3 Changed 16 years ago by romw

Owner: changed from romw to davea

This would cause a drop in benchmark scores.

See ticket #155 for further information.

David, should we re-visit this issue?

comment:4 Changed 16 years ago by Nicolas

I think ONLY the benchmark .cpp files should be compiled with these flags.

comment:5 Changed 15 years ago by mjakubicek

OK, what's the decision? Having this only for benchmark.cpp seems to be a good tradeoff...

comment:6 Changed 15 years ago by mjakubicek

Cc: mjakubicek added

comment:7 Changed 15 years ago by mjakubicek

Half year past...ping again?

comment:8 Changed 15 years ago by mjakubicek

Version: → 6.6.37

Just switched to the 6.6a branch and those flags are *not* present there.

Note: See TracTickets for help on using tickets.