[PATCH] D113237: [RISCV] Support Zifencei extension

[Luís Marques via Phabricator via llvm-commits]

luismarques added a comment.

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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113237