[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