[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