[PATCH] D63600: [test][Driver] Fix Clang :: Driver/cl-response-file.c

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 21 11:26:22 PDT 2019


rnk added a comment.

This causes the test to fail on Windows:
http://lab.llvm.org:8011/builders/clang-x64-windows-msvc/builds/7712

There seems to be something wrong with the blamelist, so it didn't send email. I see this in the log from the previous build on that bot:

  Updating llvm to 363241 at ...

But buildbot thinks the previous build was testing r364000, so it didn't send any notifications.

`printf` seems like the wrong command to use to test backslash handling, since it interprets them as escapes. `%S` expands to a path containing backslashes, and the test logs seem consistent with that theory. I'll go ahead and revert this.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63600





More information about the cfe-commits mailing list