]> git.ipfire.org Git - thirdparty/zstd.git/commit
removed -ftrapv from tests/ debug flags
authorYann Collet <cyan@fb.com>
Fri, 1 Dec 2017 18:30:53 +0000 (10:30 -0800)
committerYann Collet <cyan@fb.com>
Fri, 1 Dec 2017 18:30:53 +0000 (10:30 -0800)
commit453fb5e830356559b8a54a5655219a874a39509e
tree9482eb3ae77e693eba3180e7f8d338049ceaf065
parentb3d76e0a94502e2f484d7495c88ca3a21d44155b
removed -ftrapv from tests/ debug flags

-ftrapv is apparently buggy for gcc.
versions >= 5 are supposed to work better,
but even then, some complaints say it's still flaky when optimizations are enabled.
I even saw a post saying it only works if one creates its own signal handler,
which would make this flag no longer transparent.

on clang, it seems to work correctly.
But we would need to add a method to selectively add flags depending on compiler.
That's too much troubles for the intended benefit
(just catch integer overflows, which we can also do using ubsan).
tests/Makefile