diff options
| author | Erich Keane <erich.keane@intel.com> | 2018-02-09 00:13:49 +0000 |
|---|---|---|
| committer | Erich Keane <erich.keane@intel.com> | 2018-02-09 00:13:49 +0000 |
| commit | fa69c71dce481176b64546b5fbc098c967164575 (patch) | |
| tree | 98196449a3966d73a9911cdf0c4329382417add7 /clang/lib/Basic/Targets/PPC.cpp | |
| parent | a37e00968eafc83bbcdc8e31a49767ea3868d762 (diff) | |
| download | bcm5719-llvm-fa69c71dce481176b64546b5fbc098c967164575.tar.gz bcm5719-llvm-fa69c71dce481176b64546b5fbc098c967164575.zip | |
Fix UBSan issue with PPC::isValidCPUName
Apparently storing the pointer to a StringLiteral as
a StringRef caused this section of code to issue a ubsan
warning. This will hopefully fix that.
llvm-svn: 324687
Diffstat (limited to 'clang/lib/Basic/Targets/PPC.cpp')
| -rw-r--r-- | clang/lib/Basic/Targets/PPC.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/clang/lib/Basic/Targets/PPC.cpp b/clang/lib/Basic/Targets/PPC.cpp index 40211dba28e..3516a738098 100644 --- a/clang/lib/Basic/Targets/PPC.cpp +++ b/clang/lib/Basic/Targets/PPC.cpp @@ -493,8 +493,7 @@ static constexpr llvm::StringLiteral ValidCPUNames[] = { }; bool PPCTargetInfo::isValidCPUName(StringRef Name) const { - const StringRef *FoundName = llvm::find(ValidCPUNames, Name); - return FoundName != std::end(ValidCPUNames); + return llvm::find(ValidCPUNames, Name) != std::end(ValidCPUNames); } void PPCTargetInfo::fillValidCPUList(SmallVectorImpl<StringRef> &Values) const { |
