[PATCH] D141666: [RISCV] Proper support of extensions Zicsr and Zifencei

[Elena Lepilkina via Phabricator via llvm-commits]

eklepilkina added a comment.

Thank you for comments!

> The key aspect this patch misses right now is the versioning issue - I is still marked as 2.0.

I can update `I` extension version or separate diffrent isa versions as you mentioned. I can fix comments and make proposed by community changes. We are interested in supporting compatibility with GCC.

> It might also be interesting to explore options that temporarly sidestep the versioning discussion and improve compatibility with GCC -march strings. e.g. an initial patch that adds zicsr and zifencei but enables it by default (or otherwise handles zicsr and zifencei as a no-op)

We internally made something similar, just added extensions in the list and ignore warnings in some places. But still these extensions are already ratified so proper support is needed to be done some time


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141666