[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.

[Eli Friedman]

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