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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
