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

Shao-Ce SUN via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Dec 5 19:56:47 PST 2021

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>.

  rG LLVM Github Monorepo



More information about the cfe-commits mailing list