1 |
Ryan Hill <dirtyepic@g.o> wrote: |
2 |
|
3 |
> If your build system sets -ffast-math or -fstrict-aliasing |
4 |
> then the user can disable this by setting -fno-fast-math |
5 |
> or -fno-strict-aliasing in their CFLAGS. |
6 |
|
7 |
Just because some flags have "counter"-flags by accident, |
8 |
this does not hold for all flags. It is more reasonable to have |
9 |
other means that the flags are not modified in the first place. |
10 |
In fact, when I first introduced adding of CFLAGS, there were |
11 |
lot of complaints that this is evil and must not be done. |
12 |
I can understand this point of view (even if I know that |
13 |
certain CFLAGS should be used with the code and I would also |
14 |
prefer to have them to find possibly hidden bugs), so we |
15 |
compromised by having an option: |
16 |
With this option everybody could live well, since users with |
17 |
special settings will not run into trouble because undesired |
18 |
flags are added, and other users could just select the USE flag |
19 |
and have the benefits of appropriately optimized code. |
20 |
Up to now, that is, when we have this IMHO needless discussion |
21 |
that an option should not be an option. |
22 |
|
23 |
I hope that this answers also the question of |
24 |
Alec Warner <antarus@g.o>: |
25 |
|
26 |
: I am confused. If you want the users to use a default set of CFLAGS |
27 |
: you should set this in your build system (autotools, cmake, whatever). |
28 |
: [...] |
29 |
: I believe the above link seems to describe what you are looking to do |
30 |
: using autotools. |
31 |
|
32 |
Technically, I have no problem to force in configure.ac that certain |
33 |
CFLAGS are used (unless somebody patches the configure.ac, of course). |
34 |
The problem is that it is not good to force this if the user disagrees |
35 |
(or maybe even unless he explicitly agrees), i.e. it should be an |
36 |
option which the user really has. (If this option should only be |
37 |
documented in some INSTALL text or in the ./configure output, |
38 |
most users do not really have this option, because they would not |
39 |
even know about it.) |
40 |
|
41 |
Best Regards |
42 |
Martin |