[PATCH] D64739: [SVE][Inline-Asm] Add support to clang for SVE inline assembly
Sander de Smalen via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 17 09:24:29 PDT 2019
sdesmalen added a comment.
Functionally the patch looks good, but the title suggests this adds full inline-asm support for SVE (which would require the ACLE types proposed in D62960 <https://reviews.llvm.org/D62960>, as well as other changes), where this patch only adds support to specify SVE registers in the clobber list.
Comment at: clang/test/CodeGen/aarch64-sve-inline-asm.c:4
+ long t;
nit: there is no reason to have a different code-style for code and tests (curly brace is on next line here).
Maybe run this through clang-format?
Comment at: clang/test/CodeGen/aarch64-sve-inline-asm.c:7
+ asm volatile(
+ "ptrue p0.d\n"
nit: The asm/instructions here don't really need to make sense (as in: they are not executed), so you can combine all three tests into one, as long as the instructions are valid and z0, p0, z31 and z15 are used.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits