Avoid -O3 in nacl as it does not play well with -W.

Also cleans up some uninitialized data I saw while investigating
the problem.

R=jkummerow@chromium.org, machenbach@chromium.org

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

Patch from Ken Mixter <kmixter@google.com>.

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16235 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent 800f9f7d
...@@ -561,13 +561,21 @@ ...@@ -561,13 +561,21 @@
'cflags!': [ 'cflags!': [
'-O0', '-O0',
'-O1', '-O1',
'-O2',
'-Os', '-Os',
], ],
'cflags': [ 'cflags': [
'-fdata-sections', '-fdata-sections',
'-ffunction-sections', '-ffunction-sections',
'-O3', ],
'conditions': [
# TODO(crbug.com/272548): Avoid -O3 in NaCl
['nacl_target_arch=="none"', {
'cflags': ['-O3'],
'cflags!': ['-O2'],
}, {
'cflags': ['-O2'],
'cflags!': ['-O3'],
}],
], ],
}], }],
['v8_optimized_debug!=0 and gcc_version==44 and clang==0', { ['v8_optimized_debug!=0 and gcc_version==44 and clang==0', {
...@@ -614,13 +622,11 @@ ...@@ -614,13 +622,11 @@
'conditions': [ 'conditions': [
['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="netbsd"', { ['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="netbsd"', {
'cflags!': [ 'cflags!': [
'-O2',
'-Os', '-Os',
], ],
'cflags': [ 'cflags': [
'-fdata-sections', '-fdata-sections',
'-ffunction-sections', '-ffunction-sections',
'-O3',
'<(wno_array_bounds)', '<(wno_array_bounds)',
], ],
'conditions': [ 'conditions': [
...@@ -630,6 +636,14 @@ ...@@ -630,6 +636,14 @@
'-fno-tree-vrp', '-fno-tree-vrp',
], ],
}], }],
# TODO(crbug.com/272548): Avoid -O3 in NaCl
['nacl_target_arch=="none"', {
'cflags': ['-O3'],
'cflags!': ['-O2'],
}, {
'cflags': ['-O2'],
'cflags!': ['-O3'],
}],
], ],
}], }],
['OS=="android"', { ['OS=="android"', {
......
...@@ -55,7 +55,7 @@ struct Effect { ...@@ -55,7 +55,7 @@ struct Effect {
Modality modality; Modality modality;
Bounds bounds; Bounds bounds;
Effect() {} Effect() : modality(DEFINITE) {}
Effect(Bounds b, Modality m = DEFINITE) : modality(m), bounds(b) {} Effect(Bounds b, Modality m = DEFINITE) : modality(m), bounds(b) {}
// The unknown effect. // The unknown effect.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment