Distutils drops -fno-strict-aliasing when CFLAGS are set

[skrah]

BPO	10847
Nosy	@tarekziade, @merwok, @skrah
Superseder	bpo-969718: BASECFLAGS are not passed to module build line

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/tarekziade'
closed_at = <Date 2011-02-04.01:12:14.566>
created_at = <Date 2011-01-06.21:21:42.324>
labels = ['type-bug', 'library']
title = 'Distutils drops -fno-strict-aliasing when CFLAGS are set'
updated_at = <Date 2011-02-04.01:12:14.565>
user = 'https://github.com/skrah'

bugs.python.org fields:

activity = <Date 2011-02-04.01:12:14.565>
actor = 'eric.araujo'
assignee = 'tarek'
closed = True
closed_date = <Date 2011-02-04.01:12:14.566>
closer = 'eric.araujo'
components = ['Distutils']
creation = <Date 2011-01-06.21:21:42.324>
creator = 'skrah'
dependencies = []
files = []
hgrepos = []
issue_num = 10847
keywords = []
message_count = 2.0
messages = ['125591', '127849']
nosy_count = 3.0
nosy_names = ['tarek', 'eric.araujo', 'skrah']
pr_nums = []
priority = 'high'
resolution = 'duplicate'
stage = 'resolved'
status = 'closed'
superseder = '969718'
type = 'behavior'
url = 'https://bugs.python.org/issue10847'
versions = ['Python 3.1', 'Python 2.7', 'Python 3.2']

[skrah]

When CFLAGS are set, distutils drops -fno-strict-aliasing (among other
flags):

$ python2.7 setup.py build
...
gcc -pthread -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -I./src -I/usr/local/include -I/usr/local/include/python2.7 -c src/gmpy.c -o build/temp.linux-x86_64-2.7/src/gmpy.o
...

$ CFLAGS="-fomit-frame-pointer" python2.7 setup.py build
...
gcc -pthread -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fomit-frame-pointer -fPIC -I./src -I/usr/local/include -I/usr/local/include/python2.7 -c src/gmpy.c -o build/temp.linux-x86_64-2.7/src/gmpy.o
src/gmpy.c: In function ‘_cmp_to_object’:
src/gmpy.c:4692: warning: dereferencing type-punned pointer will break strict-aliasing rules
...

I'm not sure if this is intentional. The documentation says:

"Compiler flags can also be supplied through setting the CFLAGS
environment variable. If set, the contents of CFLAGS will be added
to the compiler flags specified in the Setup file."

To me, this sounds as if they should be appended to the regular flags.

[merwok]

Thanks for the report. It came up on the debian-python ML recently too, and is actually already reported. Following up on the other report.