craig.topper added inline comments.

================
Comment at: llvm/lib/Target/RISCV/CMakeLists.txt:18
 tablegen(LLVM RISCVGenSubtargetInfo.inc -gen-subtarget)
+tablegen(LLVM RISCVTargetParserDef.inc -gen-riscv-target-def)
 
----------------
Should not this only be in the CMakeLists for the TargetSupport library?


================
Comment at: llvm/utils/TableGen/RISCVTargetDefEmitter.cpp:9
+//
+// This tablegen backend emits an assembly printer for the current target.
+// Note that this is currently fairly skeletal, but will grow over time.
----------------
This comment says assembly printer but that’s not what this file is.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137517

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

Reply via email to