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

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

Issue 1236863009: clang/win: Enable -Wunused-local-typedef (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: unsetupset Created 5 years, 5 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/yasm/BUILD.gn ('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 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 'genperf', 74 'genperf',
75 'genperf_libs', 75 'genperf_libs',
76 'generate_files', # Needed to generate gperf and instruction files. 76 'generate_files', # Needed to generate gperf and instruction files.
77 'genstring', 77 'genstring',
78 're2c', 78 're2c',
79 ], 79 ],
80 'variables': { 80 'variables': {
81 'clang_warning_flags': [ 81 'clang_warning_flags': [
82 # yasm passes a `const elf_machine_sym*` through `void*`. 82 # yasm passes a `const elf_machine_sym*` through `void*`.
83 '-Wno-incompatible-pointer-types', 83 '-Wno-incompatible-pointer-types',
84 # reg3264type in x86expr.c is unused.
85 '-Wno-unused-local-typedef',
84 ], 86 ],
85 }, 87 },
86 'conditions': [ 88 'conditions': [
87 ['OS=="win"', { 89 ['OS=="win"', {
88 # As of VS 2013 Update 3, building this project with /analyze hits an 90 # 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 91 # internal compiler error on elf-x86-amd64.c in release builds with
90 # the amd64_x86 compiler. This halts the build and prevents subsequent 92 # the amd64_x86 compiler. This halts the build and prevents subsequent
91 # analysis. Therefore, /analyze is disabled for this project. See this 93 # analysis. Therefore, /analyze is disabled for this project. See this
92 # bug for details: 94 # bug for details:
93 # https://connect.microsoft.com/VisualStudio/feedback/details/1014799/ internal-compiler-error-when-using-analyze 95 # https://connect.microsoft.com/VisualStudio/feedback/details/1014799/ internal-compiler-error-when-using-analyze
(...skipping 480 matching lines...) Expand 10 before | Expand all | Expand 10 after
574 'include_dirs': [ 576 'include_dirs': [
575 '<@(yasm_include_dirs)', 577 '<@(yasm_include_dirs)',
576 578
577 ], 579 ],
578 'cflags': [ 580 'cflags': [
579 '-std=gnu99', 581 '-std=gnu99',
580 ], 582 ],
581 }, 583 },
582 ], 584 ],
583 } 585 }
OLDNEW
« no previous file with comments | « third_party/yasm/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698