[PATCH] D112986: [Clang][RISCV] Restrict rvv builtin-s with zve macro-s

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: clang/utils/TableGen/RISCVVEmitter.cpp:144
   Basic = 0,
   F = 1 << 1,
   D = 1 << 2,
----------------
Drop F and D here?


================
Comment at: clang/utils/TableGen/RISCVVEmitter.cpp:1319
   ListSeparator LS(" && ");
-  if (Extents & RISCVExtension::F)
+  if (PredefinedMacros & RISCVPredefinedMacro::F)
     OS << LS << "defined(__riscv_f)";
----------------
Drop F and D here


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112986