[PATCH] D32689: DEF: migrate def parser from LLD to LLVM
Rui Ueyama via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 1 19:33:41 PDT 2017
ruiu added inline comments.
================
Comment at: lib/Object/DEFParser.cpp:57-58
+
+static BumpPtrAllocator BAlloc;
+static StringSaver Saver{BAlloc};
+
----------------
martell wrote:
> ruiu wrote:
> > Remove.
> How would you suggest I allocate the StringRef when decorating `E.Name` and `E.ExtName`?
Maybe just by making `E.Name` and `E.ExtName` be std::string instead of StringRef?
Repository:
rL LLVM
https://reviews.llvm.org/D32689
More information about the llvm-commits
mailing list