Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3)

Side by Side Diff: third_party/yasm/yasm.gyp

Issue 1177153007: Use /analyze:WX- instead of /WX- - avoids 3925 D9025 warnings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Adding back libexif exclusion from /analyze Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « third_party/libexif/libexif.gyp ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 # The yasm build process creates a slew of small C subprograms that 5 # The yasm build process creates a slew of small C subprograms that
6 # dynamically generate files at various point in the build process. This makes 6 # dynamically generate files at various point in the build process. This makes
7 # the build integration moderately complex. 7 # the build integration moderately complex.
8 # 8 #
9 # There are three classes of dynamically generated files: 9 # There are three classes of dynamically generated files:
10 # 1) C source files that should be included in the build (eg., lc3bid.c) 10 # 1) C source files that should be included in the build (eg., lc3bid.c)
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
86 'conditions': [ 86 'conditions': [
87 ['OS=="win"', { 87 ['OS=="win"', {
88 # As of VS 2013 Update 3, building this project with /analyze hits an 88 # As of VS 2013 Update 3, building this project with /analyze hits an
89 # internal compiler error on elf-x86-amd64.c in release builds with 89 # internal compiler error on elf-x86-amd64.c in release builds with
90 # the amd64_x86 compiler. This halts the build and prevents subsequent 90 # the amd64_x86 compiler. This halts the build and prevents subsequent
91 # analysis. Therefore, /analyze is disabled for this project. See this 91 # analysis. Therefore, /analyze is disabled for this project. See this
92 # bug for details: 92 # bug for details:
93 # https://connect.microsoft.com/VisualStudio/feedback/details/1014799/ internal-compiler-error-when-using-analyze 93 # https://connect.microsoft.com/VisualStudio/feedback/details/1014799/ internal-compiler-error-when-using-analyze
94 'msvs_settings': { 94 'msvs_settings': {
95 'VCCLCompilerTool': { 95 'VCCLCompilerTool': {
96 'AdditionalOptions!': [ '/analyze' ] 96 'AdditionalOptions!': [ '/analyze:WX-' ]
97 }, 97 },
98 }, 98 },
99 }], 99 }],
100 ], 100 ],
101 'sources': [ 101 'sources': [
102 'source/patched-yasm/frontends/yasm/yasm-options.c', 102 'source/patched-yasm/frontends/yasm/yasm-options.c',
103 'source/patched-yasm/frontends/yasm/yasm.c', 103 'source/patched-yasm/frontends/yasm/yasm.c',
104 'source/patched-yasm/libyasm/assocdat.c', 104 'source/patched-yasm/libyasm/assocdat.c',
105 'source/patched-yasm/libyasm/bc-align.c', 105 'source/patched-yasm/libyasm/bc-align.c',
106 'source/patched-yasm/libyasm/bc-data.c', 106 'source/patched-yasm/libyasm/bc-data.c',
(...skipping 467 matching lines...) Expand 10 before | Expand all | Expand 10 after
574 'include_dirs': [ 574 'include_dirs': [
575 '<@(yasm_include_dirs)', 575 '<@(yasm_include_dirs)',
576 576
577 ], 577 ],
578 'cflags': [ 578 'cflags': [
579 '-std=gnu99', 579 '-std=gnu99',
580 ], 580 ],
581 }, 581 },
582 ], 582 ],
583 } 583 }
OLDNEW
« no previous file with comments | « third_party/libexif/libexif.gyp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698