Re: [PATCH] D12827: [analyzer] fix an error finding clang path

2015-09-12 Thread Honggyu Kim via cfe-commits
honggyu.kim added a comment. In http://reviews.llvm.org/D12827#244906, @ayartsev wrote: > Thanks, Honggyu! You're welcome, Антон! http://reviews.llvm.org/D12827 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin

Re: [PATCH] D12827: [analyzer] fix an error finding clang path

2015-09-12 Thread Антон Ярцев via cfe-commits
ayartsev added a comment. Thanks, Honggyu! http://reviews.llvm.org/D12827 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D12827: [analyzer] fix an error finding clang path

2015-09-12 Thread Ted Kremenek via cfe-commits
krememek accepted this revision. krememek added a comment. This revision is now accepted and ready to land. Applied in r247510. http://reviews.llvm.org/D12827 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mail

Re: [PATCH] D12827: [analyzer] fix an error finding clang path

2015-09-12 Thread Honggyu Kim via cfe-commits
honggyu.kim updated the summary for this revision. honggyu.kim updated this revision to Diff 34624. http://reviews.llvm.org/D12827 Files: tools/scan-build/scan-build Index: tools/scan-build/scan-build === --- tools/scan-build/scan

Re: [PATCH] D12827: [analyzer] fix an error finding clang path

2015-09-12 Thread Honggyu Kim via cfe-commits
honggyu.kim added a comment. After applying http://reviews.llvm.org/rL247466, I got the following error which was not shown before. scan-build: error: Cannot find an executable 'clang' relative to scan-build. Consider using --use-analyzer to pick a version of 'clang' to use for static analy

[PATCH] D12827: [analyzer] fix an error finding clang path

2015-09-12 Thread Honggyu Kim via cfe-commits
honggyu.kim created this revision. honggyu.kim added reviewers: ayartsev, zaks.anna, krememek. honggyu.kim added subscribers: jordan_rose, dcoughlin, cfe-commits. This patch fixes an error made by rL247466. It mistakenly wrote "-d" check in if statement with the file name itself not a directory.