[PATCH] D153742: [RISCV] Add Zce extension.

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 10:38:35 PDT 2023


asb added a comment.

Looking at the PDF, the behaviour I feel is slightly unclear is for RV64 with F and with Zce. You've chosing in this patch to allow it but to just not include zcf, though by my first reading of the spec I probably would have errored on the basis that "Specifying Zce with F includes Zca, Zcb, Zcmp, Zcmt and Zcf" and Zcf on RV64 is of course illegal. I think we have some leeway in how we handle this, but do you happen to know if GCC does something similar?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153742



More information about the llvm-commits mailing list