From e9b3495465d69615a91b0f8002f4e40ff2c14a9a Mon Sep 17 00:00:00 2001 From: Craig Topper Date: Sat, 29 Nov 2014 18:12:59 +0000 Subject: Replace std::map with std::map to handle ownership and deletion of the values. llvm-svn: 222957 --- llvm/lib/TableGen/TGParser.cpp | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) (limited to 'llvm/lib/TableGen/TGParser.cpp') diff --git a/llvm/lib/TableGen/TGParser.cpp b/llvm/lib/TableGen/TGParser.cpp index 8761726d12e..b17e8607e7f 100644 --- a/llvm/lib/TableGen/TGParser.cpp +++ b/llvm/lib/TableGen/TGParser.cpp @@ -459,12 +459,12 @@ MultiClass *TGParser::ParseMultiClassID() { return nullptr; } - MultiClass *Result = MultiClasses[Lex.getCurStrVal()]; - if (!Result) + auto it = MultiClasses.find(Lex.getCurStrVal()); + if (it == MultiClasses.end()) TokError("Couldn't find multiclass '" + Lex.getCurStrVal() + "'"); Lex.Lex(); - return Result; + return &it->second; } /// ParseSubClassReference - Parse a reference to a subclass or to a templated @@ -2290,11 +2290,13 @@ bool TGParser::ParseMultiClass() { return TokError("expected identifier after multiclass for name"); std::string Name = Lex.getCurStrVal(); - if (MultiClasses.count(Name)) + auto Result = + MultiClasses.insert(std::make_pair(Name, + MultiClass(Name, Lex.getLoc(),Records))); + if (!Result.second) return TokError("multiclass '" + Name + "' already defined"); + CurMultiClass = &Result.first->second; - CurMultiClass = MultiClasses[Name] = new MultiClass(Name, - Lex.getLoc(), Records); Lex.Lex(); // Eat the identifier. // If there are template args, parse them. @@ -2555,8 +2557,9 @@ bool TGParser::ParseDefm(MultiClass *CurMultiClass) { // To instantiate a multiclass, we need to first get the multiclass, then // instantiate each def contained in the multiclass with the SubClassRef // template parameters. - MultiClass *MC = MultiClasses[Ref.Rec->getName()]; - assert(MC && "Didn't lookup multiclass correctly?"); + auto it = MultiClasses.find(Ref.Rec->getName()); + assert(it != MultiClasses.end() && "Didn't lookup multiclass correctly?"); + MultiClass *MC = &it->second; std::vector &TemplateVals = Ref.TemplateArgs; // Verify that the correct number of template arguments were specified. -- cgit v1.2.3