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

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 9 13:36:06 PDT 2021


nickdesaulniers added inline comments.


================
Comment at: llvm/lib/MC/MCParser/AsmParser.cpp:2505
+    std::size_t Pos = 0;
+    std::unordered_set<std::string> Locals;
+
----------------
would it be preferable to use `llvm::StringSet`?

https://llvm.org/docs/ProgrammersManual.html#llvm-adt-stringset-h


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105720



More information about the llvm-commits mailing list