[PATCH] D113237: [RISCV] Support I extension version 2.1

[Shao-Ce SUN via Phabricator via llvm-commits]

achieveartificialintelligence added a comment.

In D113237#3124232 <https://reviews.llvm.org/D113237#3124232>, @luismarques wrote:

> In D113237#3124188 <https://reviews.llvm.org/D113237#3124188>, @luismarques wrote:
>
>> Should we also support `fence.i` with RVI 2.0, without zifencei?
>
> Here's a proposed path forwards:
>
> - Patch 1: Add support for RVI 2.1 and related changes. Keep RVI 2.0 as the default.
> - Patch 2: Add warnings for uses of the removed `fence.i`, CSRs and counters when using RVI 2.0 (at least when doing so by default; arguably this should error out instead when explicitly specifying version 2.0.)
> - Patch 3: Start defaulting to RVI 2.1. IMO this should only be committed after patch 2 being in effect for a while.

I tried to make I-ext imply `zifencei`, `zicsr`, and `zihintpause` automatically. BTW `zihintpause` is implemented in D93019 <https://reviews.llvm.org/D93019>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113237