From 01d7fe760581e7a1a9b77a8f944caff7bb21f456 Mon Sep 17 00:00:00 2001 From: Nico Weber Date: Fri, 23 Aug 2019 13:40:54 +0000 Subject: [PATCH] libcxx: Make gen_link_script.py print contents only in --dryrun mode The build should generally be quiet if there are no errors, and this script has been around long enough that we can remove the log output. If we ever need to debug something with this script, we can put back the logging then. Differential Revision: https://reviews.llvm.org/D66594 llvm-svn: 369757 --- libcxx/utils/gen_link_script.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libcxx/utils/gen_link_script.py b/libcxx/utils/gen_link_script.py index 238c5747cc8f..eaa752d5ae2a 100755 --- a/libcxx/utils/gen_link_script.py +++ b/libcxx/utils/gen_link_script.py @@ -24,7 +24,8 @@ def main(): parser.add_argument("--rename", action="store_true", default=False, help="Rename the output as input so we can replace it") parser.add_argument("--input", help="Path to libc++ library", required=True) - parser.add_argument("--output", help="Path to libc++ linker script", required=True) + parser.add_argument("--output", help="Path to libc++ linker script", + required=True) parser.add_argument("libraries", nargs="+", help="List of libraries libc++ depends on") args = parser.parse_args() @@ -37,9 +38,9 @@ def main(): # Generate the linker script contents. contents = "INPUT(%s)" % ' '.join([libcxx] + public_libs) - print("GENERATING SCRIPT: '%s' as file %s" % (contents, args.output)) if args.dryrun: + print("GENERATING SCRIPT: '%s' as file %s" % (contents, args.output)) return 0 # Remove the existing libc++ symlink if it exists.