[PATCH] D79842: [clang][Driver] Correct tool search path priority

Wed Jun 17 05:53:56 PDT 2020

DavidSpickett marked 2 inline comments as done.
DavidSpickett added inline comments.

================
Comment at: clang/test/Driver/program-path-priority.c:22
+/// No gccs at all, nothing is found
+// RUN: env "PATH=" %t/clang -### -target notreal-none-elf %s 2>&1 \
+// RUN:   | FileCheck --check-prefix=NO_NOTREAL_GCC %s
----------------
MaskRay wrote:
> Nit: this may be the preference of some reviewers, but they tend to favor `| \` in the end of the last line. `|` makes it clear that it has a continuation line (and if you type the first line with `|`, your shell will wait for the next line)
Ah now I understand. Yes that looks clearer so I've done that too.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79842