r247995 - Use None instead of an explicit constructor.

Douglas Katzman via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 18 08:23:17 PDT 2015


Author: dougk
Date: Fri Sep 18 10:23:16 2015
New Revision: 247995

URL: http://llvm.org/viewvc/llvm-project?rev=247995&view=rev
Log:
Use None instead of an explicit constructor.

And remove a needless 'const' since ArrayRef is immutable.
Both as suggested by dblaikie on cfe-commits.

Modified:
    cfe/trunk/lib/Driver/ToolChains.cpp
    cfe/trunk/lib/Driver/ToolChains.h

Modified: cfe/trunk/lib/Driver/ToolChains.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.cpp?rev=247995&r1=247994&r2=247995&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/ToolChains.cpp (original)
+++ cfe/trunk/lib/Driver/ToolChains.cpp Fri Sep 18 10:23:16 2015
@@ -1171,7 +1171,7 @@ static llvm::StringRef getGCCToolchainDi
 /// triple.
 void Generic_GCC::GCCInstallationDetector::init(
     const Driver &D, const llvm::Triple &TargetTriple, const ArgList &Args,
-    const ArrayRef<std::string> ExtraTripleAliases) {
+    ArrayRef<std::string> ExtraTripleAliases) {
   llvm::Triple BiarchVariantTriple = TargetTriple.isArch32Bit()
                                          ? TargetTriple.get64BitArchVariant()
                                          : TargetTriple.get32BitArchVariant();

Modified: cfe/trunk/lib/Driver/ToolChains.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.h?rev=247995&r1=247994&r2=247995&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/ToolChains.h (original)
+++ cfe/trunk/lib/Driver/ToolChains.h Fri Sep 18 10:23:16 2015
@@ -102,8 +102,7 @@ public:
     GCCInstallationDetector() : IsValid(false) {}
     void init(const Driver &D, const llvm::Triple &TargetTriple,
               const llvm::opt::ArgList &Args,
-              const ArrayRef<std::string> ExtraTripleAliases =
-                  ArrayRef<std::string>());
+              ArrayRef<std::string> ExtraTripleAliases = None);
 
     /// \brief Check whether we detected a valid GCC install.
     bool isValid() const { return IsValid; }




More information about the cfe-commits mailing list