[PATCH] Mostly correct conditional handling for Consumed analysis

Christian Wailes chriswailes at google.com
Wed Aug 28 19:14:29 PDT 2013


  Fixed a bug involving invertConsumedUnconsumed.

  VisitBinaryTest wasn't initializing TestsFor in empty tests.  This is fine now, as it would never be accessed again later.  It could be a problem in future code though.  Due to the lack of initialization the function fell off the end of the switch and hit an illegal instruction in debug mode, causing the unit test to fail.

Hi delesley, dblaikie, aaron.ballman,

http://llvm-reviews.chandlerc.com/D1501

CHANGE SINCE LAST DIFF
  http://llvm-reviews.chandlerc.com/D1501?vs=3878&id=3882#toc

Files:
  include/clang/Analysis/Analyses/Consumed.h
  lib/Analysis/Consumed.cpp
  test/SemaCXX/warn-consumed-analysis-strict.cpp
  test/SemaCXX/warn-consumed-analysis.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1501.7.patch
Type: text/x-patch
Size: 49604 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130828/878da151/attachment.bin>


More information about the cfe-commits mailing list