From: Daniel Sanders Date: Tue, 4 Apr 2017 13:52:00 +0000 (+0000) Subject: [globalisel][tablegen] Try to make MSVC happy with r299430 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=db7ed37c7a09ce750484b38117bb4c66708a3fa7;p=platform%2Fupstream%2Fllvm.git [globalisel][tablegen] Try to make MSVC happy with r299430 Fix other cases of 'const StringRef' creeping back in at the same time. This should fix the llvm-clang-x86_64-expensive-checks-win buildbot. llvm-svn: 299433 --- diff --git a/llvm/utils/TableGen/GlobalISelEmitter.cpp b/llvm/utils/TableGen/GlobalISelEmitter.cpp index 42f26e7..f581c88 100644 --- a/llvm/utils/TableGen/GlobalISelEmitter.cpp +++ b/llvm/utils/TableGen/GlobalISelEmitter.cpp @@ -118,7 +118,7 @@ public: static OperandPlaceholder CreateMatchReference(InstructionMatcher *InsnMatcher, - const StringRef InsnVarName, const StringRef SymbolicName) { + StringRef InsnVarName, StringRef SymbolicName) { OperandPlaceholder Result(OP_MatchReference); Result.MatchReference.InsnMatcher = InsnMatcher; Result.MatchReference.InsnVarName = InsnVarName; @@ -281,7 +281,7 @@ public: /// InstructionOperandMatcher is the only subclass that can return non-null /// for this. virtual Optional - getOptionalOperand(const StringRef SymbolicName) const { + getOptionalOperand(StringRef SymbolicName) const { assert(!SymbolicName.empty() && "Cannot lookup unnamed operand"); return None; } @@ -438,7 +438,7 @@ public: } unsigned getOperandIndex() const { return OpIdx; } - std::string getOperandExpr(const StringRef InsnVarName) const { + std::string getOperandExpr(StringRef InsnVarName) const { return (InsnVarName + ".getOperand(" + llvm::to_string(OpIdx) + ")").str(); } @@ -467,7 +467,7 @@ public: /// Emit a C++ expression that tests whether the instruction named in /// InsnVarName matches all the predicate and all the operands. void emitCxxPredicateExpr(raw_ostream &OS, RuleMatcher &Rule, - const StringRef InsnVarName) const { + StringRef InsnVarName) const { OS << "(/* "; if (SymbolicName.empty()) OS << "Operand " << OpIdx; @@ -632,7 +632,7 @@ public: return None; } - const OperandMatcher &getOperand(const StringRef SymbolicName) const { + const OperandMatcher &getOperand(StringRef SymbolicName) const { OptionalOM = getOptionalOperand(SymbolicName); if (OM.hasValue()) return *OM.getValue();