[PATCH] D133863: [RISCV] Add MC support of RISCV zcmt Extension

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 25 21:10:26 PDT 2023


craig.topper requested changes to this revision.
craig.topper added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/lib/Support/RISCVISAInfo.cpp:853
 Error RISCVISAInfo::checkDependency() {
+  bool HasC = Exts.count("c") != 0;
   bool HasD = Exts.count("d") != 0;
----------------
This variable is unused.


================
Comment at: llvm/lib/Support/RISCVISAInfo.cpp:913
 
+  if (HasZcmt && (HasD || HasZcd))
+    return createStringError(
----------------
It wouldn't be incompatible with Zca + D without Zcd would it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133863



More information about the cfe-commits mailing list