[cfe-commits] r157362 - in /cfe/trunk: include/clang/AST/Expr.h include/clang/Basic/DiagnosticSemaKinds.td lib/AST/Expr.cpp lib/Sema/SemaExpr.cpp lib/Sema/SemaStmt.cpp test/Sema/unused-expr.c test/SemaCXX/reinterpret-cast.cpp test/SemaCXX/unused.

[Nico Weber]

All good now. Thanks!

On Thu, May 24, 2012 at 2:07 PM, Eli Friedman <eli.friedman at gmail.com> wrote:
> On Thu, May 24, 2012 at 9:57 AM, Nico Weber <thakis at chromium.org> wrote:
>> Hi Eli,
>>
>> someone on IRC reported that this breaks the compiler-rt build. Log at
>> http://paste.ubuntu.com/1004855/ , excerpt below:
>>
>> In file included from
>> /home/lockal/bin/llvm/projects/compiler-rt/lib/tsan/rtl/tsan_clock.h:16:
>> /home/lockal/bin/llvm/projects/compiler-rt/lib/tsan/rtl/tsan_defs.h:144:9:
>> error: expression result unused; assign into a variable to force a
>> volatile load [-Werror,-Wunused-volatile-lvalue]
>>  (void)cfg;
>>        ^~~
>> /home/lockal/bin/llvm/projects/compiler-rt/lib/tsan/rtl/tsan_defs.h:145:9:
>> error: expression result unused; assign into a variable to force a
>> volatile load [-Werror,-Wunused-volatile-lvalue]
>>  (void)stats;
>>        ^~~~~
>> /home/lockal/bin/llvm/projects/compiler-rt/lib/tsan/rtl/tsan_defs.h:146:9:
>> error: expression result unused; assign into a variable to force a
>> volatile load [-Werror,-Wunused-volatile-lvalue]
>>  (void)shadow;
>>        ^~~~~~
>>
>>
>> This fires also in the chrome build, for example here:
>>
>> ../../third_party/WebKit/Source/WTF/wtf/DynamicAnnotations.cpp:48:5:
>> error: expression result unused; assign into a variable to force a
>> volatile load [-Werror,-Wunused-volatile-lvalue]
>>    DYNAMIC_ANNOTATIONS_IMPL
>>    ^~~~~~~~~~~~~~~~~~~~~~~~
>> ../../third_party/WebKit/Source/WTF/wtf/DynamicAnnotations.cpp:39:11:
>> note: expanded from macro 'DYNAMIC_ANNOTATIONS_IMPL'
>>    (void)lineno;
>>          ^~~~~~
>
> r157414 should fix those issues; please let me know if that isn't enough.
>
> -Eli