This revision was automatically updated to reflect the committed changes.
Closed by commit rL356308: [analyzer] Teach scan-build to find clang when
installed in /usr/local/bin/ (authored by dcoughlin, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Changed p
NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.
Thx!
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59406/new/
https://reviews.llvm.org/D59406
___
cfe-commits mailing list
cfe-commits@lists.l
dcoughlin updated this revision to Diff 190931.
dcoughlin edited the summary of this revision.
dcoughlin added a comment.
Herald added a subscriber: jdoerfert.
Update to restrict the new behavior to when Xcode is present.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59406/new/
https://
dcoughlin added a comment.
That's a good point. I've updated the patch to look for 'xcrun'
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59406/new/
https://reviews.llvm.org/D59406
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
NoQ added a comment.
This should indeed take care of our relatively tricky macOS use-case! I'm a bit
worried for unexpected consequences on some of the more exotic linux setups -
who knows what `../..` is supposed to contain. Maybe restrict this lookup to
Darwin somehow? - And i guess let's eve
dcoughlin created this revision.
dcoughlin added a reviewer: NoQ.
Herald added subscribers: cfe-commits, Charusso, dkrupp, donat.nagy, Szelethus,
mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun.
Herald added a project: clang.
Change scan-build to support the scenario where scan-