[PATCH] D40170: [llvm-tblgen] - Stop using std:string in RecordKeeper.

[Zachary Turner via Phabricator via llvm-commits]

zturner added a comment.

There's been some talk about making `LLVM_OPTIMIZED_TABLEGEN` the default, but it hasn't happened for whatever reason.

As for this patch, I would actually prefer if we didn't even use `std::map`.  Is ordering important here?  Can this be changed to a `llvm::StringMap`?


https://reviews.llvm.org/D40170