diff options
| author | Guillaume Chatelet <gchatelet@google.com> | 2019-10-29 17:28:34 +0100 |
|---|---|---|
| committer | Guillaume Chatelet <gchatelet@google.com> | 2019-10-29 17:36:55 +0100 |
| commit | 1c85a2e8dc7e76761d301f9a35374e0aafc757ec (patch) | |
| tree | 7ccf48d93f8bb2d12e0fecf21625c29bc04bfb45 /clang/lib/Basic/Builtins.cpp | |
| parent | d6a7da80aae62474c15cad83d361830ffc4e2566 (diff) | |
| download | llvm-1c85a2e8dc7e76761d301f9a35374e0aafc757ec.zip llvm-1c85a2e8dc7e76761d301f9a35374e0aafc757ec.tar.gz llvm-1c85a2e8dc7e76761d301f9a35374e0aafc757ec.tar.bz2 | |
isBuiltinFunc() uses StringRef instead of const char*
Summary: This prevents a bug when passing nullptr, StringRef ctor would call strlen(nullptr).
Reviewers: vlad.tsyrklevich
Subscribers: cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D69569
Diffstat (limited to 'clang/lib/Basic/Builtins.cpp')
| -rw-r--r-- | clang/lib/Basic/Builtins.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/clang/lib/Basic/Builtins.cpp b/clang/lib/Basic/Builtins.cpp index d23c280..0cd89df 100644 --- a/clang/lib/Basic/Builtins.cpp +++ b/clang/lib/Basic/Builtins.cpp @@ -47,8 +47,7 @@ void Builtin::Context::InitializeTarget(const TargetInfo &Target, AuxTSRecords = AuxTarget->getTargetBuiltins(); } -bool Builtin::Context::isBuiltinFunc(const char *Name) { - StringRef FuncName(Name); +bool Builtin::Context::isBuiltinFunc(llvm::StringRef FuncName) { for (unsigned i = Builtin::NotBuiltin + 1; i != Builtin::FirstTSBuiltin; ++i) if (FuncName.equals(BuiltinInfo[i].Name)) return strchr(BuiltinInfo[i].Attributes, 'f') != nullptr; |