[PATCH] D105254: [RISCV] Support machine constraint "S"

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

In D105254#2871278 <https://reviews.llvm.org/D105254#2871278>, @luismarques wrote:

> In D105254#2852489 <https://reviews.llvm.org/D105254#2852489>, @luismarques wrote:
>
>> Makes sense. Let's wait for the GCC Bugzilla feedback.
>
> With 'S' now documented on the GNU side, I think this can be merged.
>
> Would we keep this constraint forever regardless of how discussions of [1] might evolve?
>
> [1] https://github.com/riscv/riscv-elf-psabi-doc/issues/197

'S' will be kept forever, independent of issue/197. 'S' is useful on its own, for explicitly specifying the access modes by inline asm (I can image that the Linux kernel will use 'S' someday).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105254/new/

https://reviews.llvm.org/D105254