[PATCH] D34290: [Driver] Do a PATH lookup when using only program name with -no-canonical-prefixes

[Hans Wennborg via Phabricator via cfe-commits]

hans added a comment.

Thanks!

Please add to the description that this is for PR9576.



================
Comment at: tools/driver/driver.cpp:58
+    SmallString<128> ExecutablePath(Argv0);
+    // Do a PATH lookup, if there are no directory components.
+    if (llvm::sys::path::filename(ExecutablePath) == ExecutablePath)
----------------
What if "clang" is in the current directory? Presumably that should then be preferred over PATH.. will we get that right with this patch?

Another way would be to check access() on Argv0, which is what we do when trying to execute anyway. Is there any downside to that?


Repository:
  rL LLVM

https://reviews.llvm.org/D34290