[PATCH] D12711: ScanDirForExecutable on Windows fails to find executables with the "exe" extension in name

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 8 16:21:06 PDT 2015


rnk added a comment.

I'm not sure I like this approach. This makes the following code do the wrong thing when foo.exe exists but foo doesn't:

  if (exists("foo"))
    // try to read it and fail

I think a better approach would be to change only can_execute to deal with PATHEXT.


Repository:
  rL LLVM

http://reviews.llvm.org/D12711





More information about the llvm-commits mailing list