russell.gallop added a comment.

In D70950#1850796 <https://reviews.llvm.org/D70950#1850796>, @rnk wrote:

> This broke ClangBuildAnalyzer on Windows because it has a very naive check 
> for "clang":
>  https://github.com/aras-p/ClangBuildAnalyzer/blob/master/src/main.cpp#L148
>  I was wondering why this wasn't working on Windows anymore. =/


I think this should be fixed by this recent change to ClangBuildAnalyzer: 
https://github.com/aras-p/ClangBuildAnalyzer/commit/7ccac789ba5846c04392755a71856b18b7f0c2c1

That should cope with both clang-11 and clang.exe.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70950



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D70950: Add ProcNam... Reid Kleckner via Phabricator via cfe-commits
    • [PATCH] D70950: Add Pr... Russell Gallop via Phabricator via cfe-commits

Reply via email to