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

Richard Smith richard at metafoo.co.uk
Thu Jan 17 15:18:44 PST 2013


On Thu, Jan 17, 2013 at 2:48 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
> 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

Thanks, hopefully r172765 should fix this.



More information about the cfe-commits mailing list