[PATCH] D105720: [AsmParser] Add support to LOCAL directive.

[Jian Cai via Phabricator via llvm-commits]

jcai19 added inline comments.


================
Comment at: llvm/lib/MC/MCParser/AsmParser.cpp:2505
+    std::size_t Pos = 0;
+    std::unordered_set<std::string> Locals;
+
----------------
nickdesaulniers wrote:
> would it be preferable to use `llvm::StringSet`?
> 
> https://llvm.org/docs/ProgrammersManual.html#llvm-adt-stringset-h
Sounds good! Thanks.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105720/new/

https://reviews.llvm.org/D105720