khchen added a comment.

RISCV supports `-mcpu` with default empty arch to align gcc's `-mtune` behavior 
since clang didn't support `-mtune` before. But now clang has `-mtune`, is it a 
good idea to remove those options? (ex. `rocket-rv32/rv64`, `sifive-7-rv32/64`)



================
Comment at: clang/test/Driver/riscv-cpus.c:82
+// Check interaction between mcpu and mtune.
+//
+// RUN: %clang -target riscv32 -### -c %s 2>&1 -mcpu=sifive-e31 
-mtune=sifive-e76 | FileCheck -check-prefix=MTUNE-E31-MCPU-E76 %s
----------------
maybe we can describe what is expected interaction behavior somewhere.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89025

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

Reply via email to