Revision: 10427
Author:   [email protected]
Date:     Wed Jan 18 02:17:03 2012
Log:      Provide a switch to specify -fno-strict-aliasing

BUG=v8:1887

Review URL: https://chromiumcodereview.appspot.com/9231019
http://code.google.com/p/v8/source/detail?r=10427

Modified:
 /branches/bleeding_edge/Makefile
 /branches/bleeding_edge/build/common.gypi

=======================================
--- /branches/bleeding_edge/Makefile    Mon Jan 16 05:29:12 2012
+++ /branches/bleeding_edge/Makefile    Wed Jan 18 02:17:03 2012
@@ -87,6 +87,10 @@
 ifeq ($(presubmit), no)
   TESTFLAGS += --no-presubmit
 endif
+# strictaliasing=off (workaround for GCC-4.5)
+ifeq ($(strictaliasing), off)
+  GYPFLAGS += -Dv8_no_strict_aliasing=1
+endif

 # ----------------- available targets: --------------------
 # - "dependencies": pulls in external dependencies (currently: GYP)
=======================================
--- /branches/bleeding_edge/build/common.gypi   Tue Jan 17 06:35:38 2012
+++ /branches/bleeding_edge/build/common.gypi   Wed Jan 18 02:17:03 2012
@@ -73,6 +73,9 @@
     # Enable profiling support. Only required on Windows.
     'v8_enable_prof%': 0,

+    # Some versions of GCC 4.5 seem to need -fno-strict-aliasing.
+    'v8_no_strict_aliasing%': 0,
+
# Chrome needs this definition unconditionally. For standalone V8 builds,
     # it's handled in build/standalone.gypi.
     'want_separate_host_toolset%': 1,
@@ -227,7 +230,10 @@
             'cflags': [ '-m32' ],
             'ldflags': [ '-m32' ],
           }],
-        ],
+          [ 'v8_no_strict_aliasing==1', {
+            'cflags': [ '-fno-strict-aliasing' ],
+          }],
+        ],  # conditions
       }],
       ['OS=="solaris"', {
         'defines': [ '__C99FEATURES__=1' ],  # isinf() etc.

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to