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

Jian Cai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 16 16:02:12 PDT 2021


jcai19 updated this revision to Diff 359470.
jcai19 added a comment.

On second thought, my earlier analysis regarding the time complexity was not correct. Integrating the implementation of LOCAL directive into the for loop that handles macro arguments does not make it more expensive. This implements LOCAL directive within the for loop and avoids copying the macro body, and hopefully is more consistent with the existing code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105720

Files:
  llvm/lib/MC/MCParser/AsmParser.cpp
  llvm/test/MC/AsmParser/directive_local_alt_macro_mode.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D105720.359470.patch
Type: text/x-patch
Size: 10061 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210716/37783113/attachment-0001.bin>


More information about the llvm-commits mailing list