[PATCH] D153008: [RISCV] Allow slash-star comments in instruction operands
Abel Bernabeu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jun 16 09:46:01 PDT 2023
abel-bernabeu added a comment.
In D153008#4428568 <https://reviews.llvm.org/D153008#4428568>, @MaskRay wrote:
> This new update still applies many unneeded `getParser().Lex();` and adds a test at a wrong layer <https://maskray.me/blog/2021-08-08-toolchain-testing#the-test-checks-at-the-wrong-layer> (clang/test/CodeGen):
Would you rather have:
- LLVM IR as input
- assembly as output
- the test placed under llvm/test/CodeGen/RISCV/
?
That would make the test for verbose, but it would make the test simple. Please confirm your preference.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153008/new/
https://reviews.llvm.org/D153008
More information about the cfe-commits
mailing list