diff options
author | Anna Henningsen <anna@addaleax.net> | 2017-04-13 22:31:44 +0200 |
---|---|---|
committer | Refael Ackermann <refack@gmail.com> | 2017-04-13 18:41:25 -0400 |
commit | ea44b8b283a7b056ebf194e4187e393bbd1ae651 (patch) | |
tree | f9b1d0ee6153bdffb86c80ff8bef04b90c8a6a16 /node.gypi | |
parent | 3d2181c5f089091a6844266f7819485c0869aab2 (diff) | |
download | android-node-v8-ea44b8b283a7b056ebf194e4187e393bbd1ae651.tar.gz android-node-v8-ea44b8b283a7b056ebf194e4187e393bbd1ae651.tar.bz2 android-node-v8-ea44b8b283a7b056ebf194e4187e393bbd1ae651.zip |
build: disable -O3 for C++ coverage
The `cflags` for `--coverage` included `-O0` so far, but that was
overridden by a later `-O3`. Resolve that by adding
`'cflags!': [ '-O3' ]` and increase coverage accuracy.
Ref: https://coverage.nodejs.org/
PR-URL: https://github.com/nodejs/node/pull/12406
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'node.gypi')
-rw-r--r-- | node.gypi | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -325,7 +325,8 @@ '-O0' ], 'cflags': [ '--coverage', '-g', - '-O0' ] + '-O0' ], + 'cflags!': [ '-O3' ] }], [ 'OS=="sunos"', { 'ldflags': [ '-Wl,-M,/usr/lib/ld/map.noexstk' ], |