[PATCH] D96586: [analyzer][CTU][NFC] Add an extra regression test

2021-03-10 Thread Balázs Benics via Phabricator via cfe-commits
steakhal added a comment. In D96586#2616292 , @thakis wrote: > The test doesn't pass: http://45.33.8.238/linux/41341/step_7.txt > > I think you just need to say `rm -rf` instead of `rm -r` on line 3 (?) I hope a94ac467c2974d9afe68f3fe6cff27bd19bcfad0

[PATCH] D96586: [analyzer][CTU][NFC] Add an extra regression test

2021-03-10 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. The test doesn't pass: http://45.33.8.238/linux/41341/step_7.txt I think you just need to say `rm -rf` instead of `rm -r` on line 3 (?) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96586/new/ https://reviews.llvm.org/D9658

[PATCH] D96586: [analyzer][CTU][NFC] Add an extra regression test

2021-03-10 Thread Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG0e0ea9ffb802: [analyzer][CTU][NFC] Add an extra regression test (authored by Balazs Benics ). Repository: rG LLVM

[PATCH] D96586: [analyzer][CTU][NFC] Add an extra regression test

2021-03-08 Thread Gabor Marton via Phabricator via cfe-commits
martong accepted this revision. martong added a comment. Thanks for the test! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96586/new/ https://reviews.llvm.org/D96586 ___ cfe-commits mailing list cfe-com

[PATCH] D96586: [analyzer][CTU][NFC] Add an extra regression test

2021-03-04 Thread Balázs Benics via Phabricator via cfe-commits
steakhal added a comment. In D96586#2601856 , @NoQ wrote: >> `-analyzer-opt-analyze-headers` > > I'm actually shocked that we provide such option at all. And that it's the > first option in the list in `scan-build --help`. And that I haven't noticed > it

[PATCH] D96586: [analyzer][CTU][NFC] Add an extra regression test

2021-03-03 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. > `-analyzer-opt-analyze-headers` I'm actually shocked that we provide such option at all. And that it's the first option in the list in `scan-build --help`. And that I haven't noticed it until now

[PATCH] D96586: [analyzer][CTU][NFC] Add an extra regression test

2021-02-26 Thread Balázs Benics via Phabricator via cfe-commits
steakhal added a comment. ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96586/new/ https://reviews.llvm.org/D96586 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-

[PATCH] D96586: [analyzer][CTU][NFC] Add an extra regression test

2021-02-12 Thread Balázs Benics via Phabricator via cfe-commits
steakhal created this revision. steakhal added reviewers: martong, balazske, Szelethus, NoQ, vsavchenko, gamesh411. Herald added subscribers: ASDenysPetrov, Charusso, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware, xazax.hun, whisperity. steakhal requested r