Committed as r189594.

  -DeLesley

On Wed, Aug 28, 2013 at 7:14 PM, Christian Wailes
<[email protected]> wrote:
>   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



-- 
DeLesley Hutchins | Software Engineer | [email protected] | 505-206-0315

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to