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

Side by Side Diff: chrome/test/data/nacl/BUILD.gn

Issue 2343063002: ThinLTO: workaround LLVM bug related to -nostdlib. (Closed)
Patch Set: Created 4 years, 3 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 | « no previous file | 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 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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 import("//build/config/features.gni") 5 import("//build/config/features.gni")
6 import("//build/config/nacl/config.gni") 6 import("//build/config/nacl/config.gni")
7 import("//build/toolchain/toolchain.gni")
7 import("//ppapi/native_client/nacl_test_data.gni") 8 import("//ppapi/native_client/nacl_test_data.gni")
8 9
9 group("nacl") { 10 group("nacl") {
10 newlib = "//build/toolchain/nacl:clang_newlib_${target_cpu}" 11 newlib = "//build/toolchain/nacl:clang_newlib_${target_cpu}"
11 pnacl = "//build/toolchain/nacl:newlib_pnacl" 12 pnacl = "//build/toolchain/nacl:newlib_pnacl"
12 data_deps = [ 13 data_deps = [
13 ":exit_status_test($newlib)", 14 ":exit_status_test($newlib)",
14 ":exit_status_test($pnacl)", 15 ":exit_status_test($pnacl)",
15 ":extension_validation_cache($newlib)", 16 ":extension_validation_cache($newlib)",
16 ":irt_exception_test($newlib)", 17 ":irt_exception_test($newlib)",
(...skipping 529 matching lines...) Expand 10 before | Expand all | Expand 10 after
546 # Optimizers may translate the original code to code which 547 # Optimizers may translate the original code to code which
547 # requires builtin functions and/or relocations. Specifically, 548 # requires builtin functions and/or relocations. Specifically,
548 # the LLVM's optimizer translates for-loop based zero 549 # the LLVM's optimizer translates for-loop based zero
549 # clear to memset. 550 # clear to memset.
550 "-O0", 551 "-O0",
551 552
552 # ARM GCC emits symbols like __aeabi_unwind_cpp_pr0 in 553 # ARM GCC emits symbols like __aeabi_unwind_cpp_pr0 in
553 # .exidx sections without this flag. 554 # .exidx sections without this flag.
554 "-fno-unwind-tables", 555 "-fno-unwind-tables",
555 ] 556 ]
557
558 # Workaround against LLVM bug:
Mark Seaborn 2016/09/16 17:30:46 Can you put the explanation that's in the commit m
krasin1 2016/09/16 17:54:37 Done.
559 # https://llvm.org/bugs/show_bug.cgi?id=30403
560 if (use_thin_lto) {
561 ldflags = [ "-Wl,-plugin-opt,O0" ]
562 }
556 } 563 }
557 564
558 executable("nonsfi_libc_free_nexe") { 565 executable("nonsfi_libc_free_nexe") {
559 output_name = "libc_free_$arch" 566 output_name = "libc_free_$arch"
560 output_extension = "nexe" 567 output_extension = "nexe"
561 sources = [ 568 sources = [
562 "nonsfi/libc_free.c", 569 "nonsfi/libc_free.c",
563 ] 570 ]
564 571
565 # Here, we would like to link a relocatable, libc-free executable. 572 # Here, we would like to link a relocatable, libc-free executable.
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
605 "nonsfi/libc_free.nmf", 612 "nonsfi/libc_free.nmf",
606 ] 613 ]
607 outputs = [ 614 outputs = [
608 "${root_build_dir}/nacl_test_data/libc-free/{{source_file_part}}", 615 "${root_build_dir}/nacl_test_data/libc-free/{{source_file_part}}",
609 ] 616 ]
610 deps = [ 617 deps = [
611 ":nonsfi_libc_free_nexe", 618 ":nonsfi_libc_free_nexe",
612 ] 619 ]
613 } 620 }
614 } 621 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698