Jim added a comment.

This patch is only for RISCV.
The title should have prefix "[RISCV]".



================
Comment at: clang/lib/Driver/ToolChains/Arch/RISCV.cpp:585
+                                const llvm::opt::ArgList &Args,
+                                llvm::opt::ArgStringList &CmdArgs) {
+  // -mabi is not encoded in bitcode so we need to pass it to LTO code
----------------
Argument list should be aligned.



================
Comment at: clang/lib/Driver/ToolChains/Arch/RISCV.h:31
+                                const llvm::opt::ArgList &Args,
+                                llvm::opt::ArgStringList &CmdArgs);
 } // end namespace riscv
----------------
Argument list should be aligned.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71387



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

Reply via email to