thakis added a comment.

In D76768#1974044 <https://reviews.llvm.org/D76768#1974044>, @thakis wrote:

> If I add scan-build to clang/test/CMakeLists.txt and undo my other changes, 
> the test passes for me. I guess you always build that target, which is why 
> you didn't see this problem. But I wonder why this isn't needed on 
> non-Windows.


On non-Windows, %scan-build expands to clang/tools/scan-build/bin/scan-build 
directly, and so non-Windows doesn't require scan-build on the PATH (i.e. in 
bin/ in the build dir) since there's no scan-build.bat trampoline.

I wonder if %scan-build should expand to "perl 
clang/tools/scan-build/bin/scan-build" on Windows, instead of to 
scan-build.bat, so that the Windows setup is more similar to the non-Windows 
setup.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76768



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to