[yoshimi] Re: ...another minor itch

  • From: Will Godfrey <willgodfrey@xxxxxxxxxxxxxxx>
  • To: yoshimi@xxxxxxxxxxxxx
  • Date: Sun, 27 May 2018 20:05:47 +0100

On Sun, 27 May 2018 17:30:54 +0200
Ichthyostega <prg@xxxxxxxxxxxxxxx> wrote:

Hello Will,

just another minor problem I noticed when setting yoshimi as a local dev 
project
to be able to step through with the debugger.

The CMake-Build defines a "BuildForDebug" toggle, but does not actually use the
corresponding flags defined in "BuildOptionsDebug".

And while we at it, I'd also propose to include the flags
"-ffast-math -fno-finite-math-only" in the default for "BuildOptionsDebug".

My resoning is: --fast-math allows to emit code which behaves (for performance
reasons) slightly different with respect to the IEEE floating point standard.
And I think, it would be pointless to build a executable for debugging, which
yields slightly different (in this case totally IEEE compliant) numeric values.
This looks like a insidious pitfall for anyone investigating Yoshimi's 
behaviour
in the debugger.

What's your opinion towards this one?

Greetings,
Hermann

I can see your point here, and it seems reasonable. However, from past
experience, I'm very reluctant to make changes to CMakeLists.txt just before a
release, so lets look at this again afterwards!

-- 
Will J Godfrey
http://www.musically.me.uk
Say you have a poem and I have a tune.
Exchange them and we can both have a poem, a tune, and a song.
Yoshimi source code is available from either: 
https://sourceforge.net/projects/yoshimi
Or: https://github.com/Yoshimi/yoshimi
Our list archive is at: https://www.freelists.org/archive/yoshimi
To post, email to yoshimi@xxxxxxxxxxxxx

Other related posts: