[PATCH] D117910: [RISCV] Add side-effect-free vsetvli intrinsics

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added a comment.

In D117910#3266105 <https://reviews.llvm.org/D117910#3266105>, @kito-cheng wrote:

> I would prefer use that to replace existing vsetvlimax and vsetvli once we verified that, I believe  vsetvl and vsetvlmax intrinsic should be no side effect on C/C++ language level.

Yes I would have preferred that too. However I spoke with Craig offline and he told me that there are users which have now come to depend on the C/C++ vsetvli/vsetvlimax builtins having side effects. So I fear that ship may have sailed? I'd definitely be up for discussing a transition path but we'd need to assess the implications and any transition path.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117910