maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

StringSwitch cannot be copied or moved. 

...but most importantly, it cannot be used well with 'auto', because
the inferred type is StringSwitch rather than the result type. This
is a problem because StringSwitch stores addresses of temporary
values rather than copying or moving the value into its own storage.

Changing this uncovered the bug in PassBuilder, also in this patch.
Clang doesn't seem to have any occurrences of the issue.

llvm-svn: 276652
This commit is contained in:
Jordan Rose 2016-07-25 17:08:24 +00:00
parent df4b4a8fab
commit 0cdbe7a572
2 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
llvm/include/llvm/ADT/StringSwitch.h
View File

@ -53,6 +53,13 @@ public:
explicit StringSwitch(StringRef S) explicit StringSwitch(StringRef S)
: Str(S), Result(nullptr) { } : Str(S), Result(nullptr) { }
// StringSwitch is neither copyable nor movable.
StringSwitch(const StringSwitch &) = delete;
StringSwitch(StringSwitch &&) = delete;
void operator=(const StringSwitch &) = delete;
void operator=(StringSwitch &&) = delete;
~StringSwitch() = default;
template<unsigned N> template<unsigned N>
LLVM_ATTRIBUTE_ALWAYS_INLINE LLVM_ATTRIBUTE_ALWAYS_INLINE
StringSwitch& Case(const char (&S)[N], const T& Value) { StringSwitch& Case(const char (&S)[N], const T& Value) {

2
llvm/lib/Passes/PassBuilder.cpp
View File

@ -334,7 +334,7 @@ bool PassBuilder::parseModulePassName(ModulePassManager &MPM, StringRef Name,
return false; return false;
assert(Matches.size() == 3 && "Must capture two matched strings!"); assert(Matches.size() == 3 && "Must capture two matched strings!");
auto L = StringSwitch<OptimizationLevel>(Matches[2]) OptimizationLevel L = StringSwitch<OptimizationLevel>(Matches[2])
.Case("O0", O0) .Case("O0", O0)
.Case("O1", O1) .Case("O1", O1)
.Case("O2", O2) .Case("O2", O2)