From 9052cc8ec75aaeea78f9e2825deb70587f4fa420 Mon Sep 17 00:00:00 2001 From: George Rimar Date: Thu, 23 Nov 2017 06:52:44 +0000 Subject: [PATCH] Revert r318822 "[llvm-tblgen] - Stop using std::string in RecordKeeper." It reported to have problems with memory sanitizers and DBUILD_SHARED_LIBS=ON. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@318899 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/TableGen/Record.h | 2 +- utils/TableGen/CTagsEmitter.cpp | 9 +++++---- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/include/llvm/TableGen/Record.h b/include/llvm/TableGen/Record.h index d0e6ddbb878..55b4dfe2fa2 100644 --- a/include/llvm/TableGen/Record.h +++ b/include/llvm/TableGen/Record.h @@ -1525,7 +1525,7 @@ struct MultiClass { }; class RecordKeeper { - using RecordMap = std::map>; + using RecordMap = std::map>; RecordMap Classes, Defs; public: diff --git a/utils/TableGen/CTagsEmitter.cpp b/utils/TableGen/CTagsEmitter.cpp index e72430078ba..5213cd90446 100644 --- a/utils/TableGen/CTagsEmitter.cpp +++ b/utils/TableGen/CTagsEmitter.cpp @@ -28,17 +28,18 @@ namespace { class Tag { private: - StringRef Id; + const std::string *Id; SMLoc Loc; public: - Tag(StringRef Name, const SMLoc Location) : Id(Name), Loc(Location) {} - int operator<(const Tag &B) const { return Id < B.Id; } + Tag(const std::string &Name, const SMLoc Location) + : Id(&Name), Loc(Location) {} + int operator<(const Tag &B) const { return *Id < *B.Id; } void emit(raw_ostream &OS) const { const MemoryBuffer *CurMB = SrcMgr.getMemoryBuffer(SrcMgr.FindBufferContainingLoc(Loc)); auto BufferName = CurMB->getBufferIdentifier(); std::pair LineAndColumn = SrcMgr.getLineAndColumn(Loc); - OS << Id << "\t" << BufferName << "\t" << LineAndColumn.first << "\n"; + OS << *Id << "\t" << BufferName << "\t" << LineAndColumn.first << "\n"; } }; -- 2.50.1