[PATCH] D80879: clang-tidy and clang-query wont crash with invalid command line options

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun May 31 09:01:57 PDT 2020


njames93 reopened this revision.
njames93 added a comment.
This revision is now accepted and ready to land.

In D80879#2065350 <https://reviews.llvm.org/D80879#2065350>, @thakis wrote:

> This breaks check-clang-tools on windows: http://45.33.8.238/win/16485/step_8.txt
>
> Looks like you need to optionally allow a .exe suffix in the test. Please take a look, and revert if it takes a while until you can fix.
>
> (I'm on a phone, else I'd fix myself.)


Just seen the buildbot, I've reverted it for now. Don't have windows to test so didnt want to fire away more breaking commits


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80879





More information about the cfe-commits mailing list