[PATCH] D91460: [AsmParser] make .ascii/.asciz/.string support multiple strings

Jian Cai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 13 13:59:36 PST 2020


jcai19 added a comment.

In D91460#2395030 <https://reviews.llvm.org/D91460#2395030>, @jrtc27 wrote:

> This needs tests, both to show it accepts multiple strings and to show for each of the three directives that it produces the right output (i.e. is equivalent to multiple individual directives).

Thanks for the reminder. I did some local testing but forgot to add tests and was about to leave a comment here that I would add them soon.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91460



More information about the llvm-commits mailing list