From 5640ae48db6a254bd99b6b9c052a7bf5ad610402 Mon Sep 17 00:00:00 2001 From: Rafael Espindola Date: Fri, 2 Jan 2015 22:56:15 +0000 Subject: [PATCH] Don't warn on unused -fno-lto. It is somewhat common for CFLAGS to be used with .s files. We were already ignoring -flto. This patch just does the same for -fno-lto. llvm-svn: 225093 --- clang/lib/Driver/Tools.cpp | 5 +++-- clang/test/Driver/clang-s-opts.s | 3 +++ 2 files changed, 6 insertions(+), 2 deletions(-) create mode 100644 clang/test/Driver/clang-s-opts.s diff --git a/clang/lib/Driver/Tools.cpp b/clang/lib/Driver/Tools.cpp index 90d1acc..93027df 100644 --- a/clang/lib/Driver/Tools.cpp +++ b/clang/lib/Driver/Tools.cpp @@ -3379,9 +3379,10 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA, D.Diag(diag::warn_ignored_gcc_optimization) << (*it)->getAsString(Args); } - // Don't warn about unused -flto. This can happen when we're preprocessing or - // precompiling. + // Don't warn about unused -f(no-)?lto. This can happen when we're preprocessing, + // precompiling or assembling. Args.ClaimAllArgs(options::OPT_flto); + Args.ClaimAllArgs(options::OPT_fno_lto); Args.AddAllArgs(CmdArgs, options::OPT_R_Group); Args.AddAllArgs(CmdArgs, options::OPT_W_Group); diff --git a/clang/test/Driver/clang-s-opts.s b/clang/test/Driver/clang-s-opts.s new file mode 100644 index 0000000..ccbad69 --- /dev/null +++ b/clang/test/Driver/clang-s-opts.s @@ -0,0 +1,3 @@ +// RUN: %clang -### -c -Werror -flto -fno-lto %s 2>&1 | FileCheck %s + +// CHECK-NOT: argument unused during compilation -- 2.7.4