Inline ShouldUseIntegratedAssembler and move the documentation to

useIntegratedAs.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@177300 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/include/clang/Driver/ToolChain.h b/include/clang/Driver/ToolChain.h
index b04dbe61987292519108cf5cbde702bd086af616..32b739dc63510adf65be1e25235115651d15d1ef 100644 (file)
--- a/include/clang/Driver/ToolChain.h
+++ b/include/clang/Driver/ToolChain.h
@@ -137,6 +137,7 @@ public:
   /// by default.
   virtual bool IsIntegratedAssemblerDefault() const { return false; }
 
+  /// \brief Check if the toolchain should use the integrated assembler.
   bool useIntegratedAs(const ArgList &Args) const;
 
   /// IsStrictAliasingDefault - Does this tool chain use -fstrict-aliasing by

diff --git a/lib/Driver/Tools.cpp b/lib/Driver/Tools.cpp
index 96d10f53aed85f322fd10fd30821d8c93aa784d8..a53e9f8147e4931f3ccaeb8b8d8ac324643d4f8c 100644 (file)
--- a/lib/Driver/Tools.cpp
+++ b/lib/Driver/Tools.cpp
@@ -1414,20 +1414,13 @@ static void addExceptionArgs(const ArgList &Args, types::ID InputType,
     CmdArgs.push_back("-fexceptions");
 }
 
-/// \brief Check if the toolchain should use the integrated assembler.
-static bool ShouldUseIntegratedAssembler(const ArgList &Args,
-                                         const ToolChain &TC) {
-  // FIXME: inline
-  return TC.useIntegratedAs(Args);
-}
-
 static bool ShouldDisableCFI(const ArgList &Args,
                              const ToolChain &TC) {
   bool Default = true;
   if (TC.getTriple().isOSDarwin()) {
     // The native darwin assembler doesn't support cfi directives, so
     // we disable them if we think the .s file will be passed to it.
-    Default = ShouldUseIntegratedAssembler(Args, TC);
+    Default = TC.useIntegratedAs(Args);
   }
   return !Args.hasFlag(options::OPT_fdwarf2_cfi_asm,
                        options::OPT_fno_dwarf2_cfi_asm,
@@ -1438,7 +1431,7 @@ static bool ShouldDisableDwarfDirectory(const ArgList &Args,
                                         const ToolChain &TC) {
   bool UseDwarfDirectory = Args.hasFlag(options::OPT_fdwarf_directory_asm,
                                         options::OPT_fno_dwarf_directory_asm,
-                                        ShouldUseIntegratedAssembler(Args, TC));
+                                        TC.useIntegratedAs(Args));
   return !UseDwarfDirectory;
 }
 
@@ -2813,7 +2806,7 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
   // -fmodules-autolink (on by default when modules is enabled) automatically
   // links against libraries for imported modules.  This requires the
   // integrated assembler.
-  if (HaveModules && ShouldUseIntegratedAssembler(Args, getToolChain()) &&
+  if (HaveModules && getToolChain().useIntegratedAs(Args) &&
       Args.hasFlag(options::OPT_fmodules_autolink,
                    options::OPT_fno_modules_autolink,
                    true)) {