[PATCH] D41254: [analyzer] WIP: trackNullOrUndefValue: peel off ParenImpCasts before tracking.

[Artem Dergachev via Phabricator via cfe-commits]

NoQ created this revision.
NoQ added reviewers: dcoughlin, xazax.hun, a.sidorin, george.karpenkov, szepet.
Herald added subscribers: cfe-commits, rnkovacs, eraman.

Continuing experiments in https://reviews.llvm.org/D41253, i'm looking into unwrapping parentheses and implicit casts from the tracked expression. The `peelOffOuterExpr` method already does more, so the current situation where we're not doing this when there's no explicit outer expression looks like an accidental omission; i'd try to provide actual justification for this stuff.

This gives a lot of fancy improvements in the diagnostics and suppressions:

- `uninit-const.cpp`: We unwrap an implicit `LValueToRValue` cast around a reference, which allows us to track the undefined value through the pass-by-reference function call back to the original uninitialized variable.
- `false-positive-suppression.m`: `ARCReclaimReturnedObject` is unwrapped, so the respective suppression now works correctly under Objective-C automatic reference counting.
- `inlining/path-notes.m`: By unwrapping another `LValueToRValueCast`, we display the specific diagnostic for the interesting value, instead of the generic diagnostic. It's still surprising that we don't display the same diagnostic in the non-interesting value case, so this remains to be fixed.
- `inlining/path-notes.cpp`: A **regression** here! This one is similar to `uninit-const.cpp`, but here when we unwrap `LValueToRValueCast` and dive into the function we suddenly get a //duplicate// diagnostic. Here `FindLastStoreBRVisitor` is not de-duplicated correctly. I'd explain what's going on in a follow-up patch that works around the problem, and for now i mark this test as FIXME so that all changes remain tested.


Repository:
  rC Clang

https://reviews.llvm.org/D41254

Files:
  lib/StaticAnalyzer/Core/BugReporterVisitors.cpp
  test/Analysis/inlining/false-positive-suppression.m
  test/Analysis/inlining/inline-defensive-checks.c
  test/Analysis/inlining/path-notes.cpp
  test/Analysis/inlining/path-notes.m
  test/Analysis/uninit-const.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41254.127006.patch
Type: text/x-patch
Size: 136456 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171214/0b0f5bad/attachment-0001.bin>