r283776 - Use StringRef in LangOptions::isNoBuiltinFunc API (NFC)

Mehdi Amini via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 10 09:34:07 PDT 2016


Author: mehdi_amini
Date: Mon Oct 10 11:34:07 2016
New Revision: 283776

URL: http://llvm.org/viewvc/llvm-project?rev=283776&view=rev
Log:
Use StringRef in LangOptions::isNoBuiltinFunc API (NFC)

Modified:
    cfe/trunk/include/clang/Basic/LangOptions.h
    cfe/trunk/lib/Basic/LangOptions.cpp

Modified: cfe/trunk/include/clang/Basic/LangOptions.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/LangOptions.h?rev=283776&r1=283775&r2=283776&view=diff
==============================================================================
--- cfe/trunk/include/clang/Basic/LangOptions.h (original)
+++ cfe/trunk/include/clang/Basic/LangOptions.h Mon Oct 10 11:34:07 2016
@@ -165,7 +165,7 @@ public:
 
   /// \brief Is this a libc/libm function that is no longer recognized as a
   /// builtin because a -fno-builtin-* option has been specified?
-  bool isNoBuiltinFunc(const char *Name) const;
+  bool isNoBuiltinFunc(StringRef Name) const;
 };
 
 /// \brief Floating point control options

Modified: cfe/trunk/lib/Basic/LangOptions.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/LangOptions.cpp?rev=283776&r1=283775&r2=283776&view=diff
==============================================================================
--- cfe/trunk/lib/Basic/LangOptions.cpp (original)
+++ cfe/trunk/lib/Basic/LangOptions.cpp Mon Oct 10 11:34:07 2016
@@ -36,8 +36,7 @@ void LangOptions::resetNonModularOptions
   CurrentModule.clear();
 }
 
-bool LangOptions::isNoBuiltinFunc(const char *Name) const {
-  StringRef FuncName(Name);
+bool LangOptions::isNoBuiltinFunc(StringRef FuncName) const {
   for (unsigned i = 0, e = NoBuiltinFuncs.size(); i != e; ++i)
     if (FuncName.equals(NoBuiltinFuncs[i]))
       return true;




More information about the cfe-commits mailing list