[PATCH] D105274: Define sys::getHostCPUName for RISC-V

[Andreas Schwab via Phabricator via llvm-commits]

schwab added inline comments.


================
Comment at: llvm/lib/Support/Host.cpp:1325
+  return "generic-rv64";
+#else
+  return "generic-rv32";
----------------
craig.topper wrote:
> Should we check __riscv_xlen == 32 here and leave some assert return "generic" if its not one of the 2 values. That would cover us better if __riscv_xlen isn't defined for some reason
What do you want to assert?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105274