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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to