[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