[PATCH] D154931: [LoongArch] Support InlineAsm for LSX and LASX
WÁNG Xuěruì via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Jul 23 23:19:20 PDT 2023
xen0n accepted this revision.
xen0n added a comment.
Seems the `u` and `w` modifiers are strictly an implementation detail that's different from ordinary inline asm constraints, so even though they take up precious single-letter namespace I'd be fine.
Given the architecture-specific nature of the change and usefulness for downstream users looking to add SIMD to their programs, I'd be okay with the change. Thanks for your contribution.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154931/new/
https://reviews.llvm.org/D154931
More information about the cfe-commits
mailing list