In most of cases, it has a single space after comma in assembly operands.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Unit Tests
Event Timeline
Comment Actions
The examples in the V spec show no spaces. Are you going to update that too? What about binutils?
Comment Actions
I refer to https://github.com/riscv/riscv-binutils-gdb/blob/c5ef83e53e2528c0737bb0f40e6b77dc8b82eac8/gas/testsuite/gas/riscv/vector-insns.s
It looks that binutils can accept two formats with or without spaces.
I pull a request to update riscv-v-spec.
https://github.com/riscv/riscv-v-spec/pull/693
Comment Actions
Note that I pushed rG292f4197249b to fix the failing tests introduced by this patch. Please check the test results before merging in the future, e.g. https://reviews.llvm.org/B107914 was showing this issue.