[PATCH] D62899: [analyzer][UninitializedObjectChecker] Mark uninitialized regions as interesting.

2019-08-20 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. In D62899#1634657 , @Szelethus wrote: > In D62899#1551312 , @NoQ wrote: > > > In D62899#1549715 , @Szelethus > > wrote: > > > > > Added a proper

[PATCH] D62899: [analyzer][UninitializedObjectChecker] Mark uninitialized regions as interesting.

2019-08-18 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. In D62899#1551312 , @NoQ wrote: > In D62899#1549715 , @Szelethus wrote: > > > Added a proper testfile. The only downside of it is that it doesn't test > > anything. > > > Use creduce!

[PATCH] D62899: [analyzer][UninitializedObjectChecker] Mark uninitialized regions as interesting.

2019-06-19 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. In D62899#1549715 , @Szelethus wrote: > Added a proper testfile. The only downside of it is that it doesn't test > anything. Use creduce! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62899/new/

[PATCH] D62899: [analyzer][UninitializedObjectChecker] Mark uninitialized regions as interesting.

2019-06-19 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus updated this revision to Diff 205511. Szelethus added a comment. Added a proper testfile. The only downside of it, is that it doesn't test anything. Literally nothing would change is I didn't mark the fields interesting. I'll take this back to the drawing board. CHANGES SINCE LAST

[PATCH] D62899: [analyzer][UninitializedObjectChecker] Mark uninitialized regions as interesting.

2019-06-16 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. In D62899#1544630 , @NoQ wrote: > I don't remember what exactly does `markInteresting()` do and these tests > don't really convince me that it does anything at all. Halp? >.< Damnit, forgot my actual test file in the office

[PATCH] D62899: [analyzer][UninitializedObjectChecker] Mark uninitialized regions as interesting.

2019-06-14 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. I don't remember what exactly does `markInteresting()` do and these tests don't really convince me that it does anything at all. Halp? >.< Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62899/new/ https://reviews.llvm.org/D62899

[PATCH] D62899: [analyzer][UninitializedObjectChecker] Mark uninitialized regions as interesting.

2019-06-14 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. Ping, anything against this? :) Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62899/new/ https://reviews.llvm.org/D62899 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D62899: [analyzer][UninitializedObjectChecker] Mark uninitialized regions as interesting.

2019-06-12 Thread Balogh, Ádám via Phabricator via cfe-commits
baloghadamsoftware added a comment. Looks much better than the original one, but why did the warning move to the correct place just because marking the region as interesting? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62899/new/

[PATCH] D62899: [analyzer][UninitializedObjectChecker] Mark uninitialized regions as interesting.

2019-06-05 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus created this revision. Szelethus added reviewers: NoQ, xazax.hun, dcoughlin, baloghadamsoftware, rnkovacs. Szelethus added a project: clang. Herald added subscribers: cfe-commits, Charusso, gamesh411, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, szepet, whisperity. Repository: rC