Skip to content

Commit

Permalink
Use -O3 instead of -Ofast
Browse files Browse the repository at this point in the history
Because we use NAN in a few places in the code at the moment, so
using -ffast-math could be an issue, and new release of clang
complains about it.

Would still like to avoid using NAN or INFINITY in places where
we don't really have to.
  • Loading branch information
guillaumechereau committed Aug 11, 2024
1 parent 9e79543 commit cde28f3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion SConstruct
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ if env['werror']:
env.Append(CCFLAGS='-Werror')

if env['mode'] not in ['debug', 'analyze']:
env.Append(CPPDEFINES='NDEBUG', CCFLAGS='-Ofast')
env.Append(CPPDEFINES='NDEBUG', CCFLAGS='-O3')

if env['mode'] == 'debug':
env.Append(CCFLAGS=['-O0'])
Expand Down

0 comments on commit cde28f3

Please sign in to comment.