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

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

Issue 1269293003: Explicitly pass -Wno-unused-function to the targets that need it. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: gnnnnnnnnnnn Created 5 years, 4 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 540 matching lines...) Expand 10 before | Expand all | Expand 10 after
551 'source/patched-yasm/tools/re2c/substr.c', 551 'source/patched-yasm/tools/re2c/substr.c',
552 'source/patched-yasm/tools/re2c/translate.c', 552 'source/patched-yasm/tools/re2c/translate.c',
553 ], 553 ],
554 'include_dirs': [ 554 'include_dirs': [
555 '<@(yasm_include_dirs)', 555 '<@(yasm_include_dirs)',
556 ], 556 ],
557 'cflags': [ 557 'cflags': [
558 '-std=gnu99', 558 '-std=gnu99',
559 ], 559 ],
560 'variables': { 560 'variables': {
561 'clang_warning_flags': [
561 # re2c is missing CLOSEVOP from one switch. 562 # re2c is missing CLOSEVOP from one switch.
562 'clang_warning_flags': [ '-Wno-switch' ], 563 '-Wno-switch',
564 # re2c contains many static functions in headers (because it's
565 # a C library predating C99.)
566 '-Wno-unused-function',
567 ],
563 }, 568 },
564 'msvs_disabled_warnings': [ 4267 ], 569 'msvs_disabled_warnings': [ 4267 ],
565 }, 570 },
566 { 571 {
567 'target_name': 'genmodule', 572 'target_name': 'genmodule',
568 'type': 'executable', 573 'type': 'executable',
569 'toolsets': ['host'], 574 'toolsets': ['host'],
570 'dependencies': [ 575 'dependencies': [
571 'config_sources', 576 'config_sources',
572 ], 577 ],
573 'sources': [ 578 'sources': [
574 'source/patched-yasm/libyasm/genmodule.c', 579 'source/patched-yasm/libyasm/genmodule.c',
575 ], 580 ],
576 'include_dirs': [ 581 'include_dirs': [
577 '<@(yasm_include_dirs)', 582 '<@(yasm_include_dirs)',
578 583
579 ], 584 ],
580 'cflags': [ 585 'cflags': [
581 '-std=gnu99', 586 '-std=gnu99',
582 ], 587 ],
583 }, 588 },
584 ], 589 ],
585 } 590 }
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