From: Ben Noordhuis <info@bnoordhuis.nl>
Date: Mon, 4 May 2015 15:08:15 +0000 (+0200)
Subject: build: disable -Og when building with clang
X-Git-Tag: v2.0.0~6
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e67542ae17510e3657c2a946fde3dee4d775ac88;p=platform%2Fupstream%2Fnodejs.git

build: disable -Og when building with clang

clang does not yet support -Og, fall back to -O0.

See: https://llvm.org/bugs/show_bug.cgi?id=20765
Fixes: https://github.com/iojs/io.js/issues/1608
PR-URL: https://github.com/iojs/io.js/pull/1609
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
---

diff --git a/common.gypi b/common.gypi
index 36a4161..385c3cd 100644
--- a/common.gypi
+++ b/common.gypi
@@ -54,8 +54,13 @@
           'v8_enable_handle_zapping%': 1,
         },
         'defines': [ 'DEBUG', '_DEBUG' ],
-        'cflags': [ '-g', '-Og' ],
+        'cflags': [ '-g' ],
         'conditions': [
+          ['clang==1', {
+            'cflags': [ '-O0' ],  # https://llvm.org/bugs/show_bug.cgi?id=20765
+          }, {
+            'cflags': [ '-Og' ],  # Debug-friendly optimizations only.
+          }],
           ['target_arch=="x64"', {
             'msvs_configuration_platform': 'x64',
           }],