[PATCH] D67508: [RISCV] support mutilib in baremetal environment

Kito Cheng via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 17 15:33:45 PDT 2019


kito-cheng added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Gnu.cpp:1548
+                {M.gccSuffix(),
+                 "/../../../../riscv64-unknown-elf/lib" + M.gccSuffix()});
+          });
----------------
It could be "riscv32-unknown-elf" other than "riscv64-unknown-elf".


================
Comment at: clang/lib/Driver/ToolChains/RISCVToolchain.cpp:129
+  }
+
   std::string Linker = getToolChain().GetProgramPath(getShortName());
----------------
This change seems like unrelated to multi-lib, could you split this change into new patch and add a test for that?


================
Comment at: clang/lib/Driver/ToolChains/RISCVToolchain.cpp:132
 
   if (D.isUsingLTO()) {
     assert(!Inputs.empty() && "Must have at least one input.");
----------------
This part will conflict when apply patch, you might generate patch with https://reviews.llvm.org/D67409, could you rebase the patch with current trunk?


Repository:
  rC Clang

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

https://reviews.llvm.org/D67508





More information about the cfe-commits mailing list