[PATCH] D153008: [RISCV] Allow slash-star comments in instruction operands
Abel Bernabeu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 20 02:36:46 PDT 2023
abel-bernabeu updated this revision to Diff 532837.
abel-bernabeu added a comment.
I just this flashback...
For code coverage of one of the Lex() calls in ParseInstruction we needed one case
where an instruction has no operands.
We had, not one, but three. Am keeping just the "nop" case, which is the simplest
and most known instruction and moving it to the top of the test suite, so the tests
are listed in increasing complexity (from the simplest to reach test points to
the most sophisticated and least frequently used parser features).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153008/new/
https://reviews.llvm.org/D153008
Files:
llvm/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp
llvm/test/MC/RISCV/comments-zdinx.ll
llvm/test/MC/RISCV/comments.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D153008.532837.patch
Type: text/x-patch
Size: 10420 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230620/7497c726/attachment-0001.bin>
More information about the cfe-commits
mailing list