[PATCH] D148066: [RISCV] Add Smaia and Ssaia extensions support

Alex Bradbury via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 12 11:40:14 PDT 2023


asb added a comment.

In D148066#4262379 <https://reviews.llvm.org/D148066#4262379>, @reames wrote:

> I would be fine landing this as experimental before ratification.  I see no real downside to doing that

My concern would be that as we don't gate CSR names on enabling the relevant extension, people could start using CSR names and encodings that could change, without opting in via `-menable-experimental-extensions`, perhaps not realising that they're using the unratified version. OTOH, you could argue it was user error from the start by not trying to specify all the needed extensions in the ISA naming string.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148066



More information about the cfe-commits mailing list