commit c0591ca026144230f08b0c8d2c9b8f47e039fca4
author mtklein <mtklein@google.com> Fri Nov 21 14:42:35 2014 -0800
committer Commit bot <commit-bot@chromium.org> Fri Nov 21 14:42:35 2014 -0800
tree b6f9ee04463f5bd0599354b4809bbe24660fea46
parent 77d77f446d1685cd21465627a747341c3c3665e4

Revert of Add -ftrapv to debug builds. (patchset #1 id:1 of https://codereview.chromium.org/702893003/)

Reason for revert:
Not clean yet.

Original issue's description:
> Add -ftrapv to debug builds.
>
> This adds instrumentation to check that signed integers never under- or
> overflow, which is undefined behavior.
>
> BUG=skia:3096
>
> Committed: https://skia.googlesource.com/skia/+/ebacb6127a952910f43a59460af43427d93df46c

TBR=bungeman@google.com,mtklein@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:3096

Review URL: https://codereview.chromium.org/751713002

gyp/common_conditions.gypi

diff --git a/gyp/common_conditions.gypi b/gyp/common_conditions.gypi
index 1684151..059806f 100644
--- a/gyp/common_conditions.gypi
+++ b/gyp/common_conditions.gypi
@@ -414,7 +414,6 @@
             'ldflags': ['--coverage'],
           },
           'Debug': {
-            'cflags': ['-ftrapv'],  # Crash on signed integer over/underflow.
           },
           'Release': {
             'cflags': [
@@ -595,7 +594,7 @@
         ],
         'configurations': {
           'Debug': {
-            'cflags': ['-g', '-ftrapv']
+            'cflags': ['-g']
           },
           'Release': {
             'cflags': ['-O2'],