[cfe-commits] r172760 - in /cfe/trunk: lib/Sema/SemaChecking.cpp test/SemaCXX/warn-unsequenced.cpp

[Dmitri Gribenko]

On Fri, Jan 18, 2013 at 12:06 AM, Richard Smith
<richard-llvm at metafoo.co.uk> wrote:
> Author: rsmith
> Date: Thu Jan 17 16:06:26 2013
> New Revision: 172760
>
> URL: http://llvm.org/viewvc/llvm-project?rev=172760&view=rev
> Log:
> -Wunsequenced: if the LHS of an &&, || or ?: is not constant, check for
> unsequenced operations in the RHS. We don't compare the RHS with the rest of
> the expression yet; such checks will need care to avoid diagnosing unsequenced
> operations which are both in conditionally-evaluated subexpressions which
> actually can't occur together, such as in '(b && ++x) + (!b && ++x)'.

Hello Richard,

This commit causes a stack overflow on
test/Index/annotate-deep-statements.cpp on Linux/x86-64.

http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/432

Dmitri

-- 
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/