MaskRay marked an inline comment as done.
MaskRay added inline comments.

================
Comment at: clang/lib/Driver/ToolChain.cpp:556-557
+  // -fld-path= takes precedence over -fuse-ld= and specifies the executable
+  // name. PATH is consulted if the value does not contain /. -B is
+  // intentionally ignored.
+  if (const Arg *A = Args.getLastArg(options::OPT_fld_path_EQ)) {
----------------
jyknight wrote:
> Shouldn't this use -B paths? Why do we want to ignore them?
If the value is relative, respect -Bprefix and COMPILER_PATH? (COMPILER is 
currently not well supported in clang IIUC)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83015



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D83015: [Driver] Add ... Fangrui Song via Phabricator via cfe-commits

Reply via email to