[PATCH] D126192: [Driver] Support linking with lld for target AVR

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 26 17:45:59 PDT 2022


MaskRay added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/AVR.cpp:435
 
-  std::string Linker = getToolChain().GetProgramPath(getShortName());
+  // Compute the linker program path, and use GNU "avr-ld" as default.
+  const Arg* A = Args.getLastArg(options::OPT_fuse_ld_EQ);
----------------
You can just call `ToolChain.GetLinkerPath()`. I don't think checking `can_execute` is necessary.


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

https://reviews.llvm.org/D126192



More information about the cfe-commits mailing list