[llvm-commits] [llvm] r121904 - /llvm/trunk/lib/MC/MCStreamer.cpp

[Matt Beaumont-Gay]

On Wed, Dec 15, 2010 at 14:57, Eric Christopher <echristo at apple.com> wrote:
>
> On Dec 15, 2010, at 2:21 PM, Matt Beaumont-Gay wrote:
>
> Author: matthewbg
> Date: Wed Dec 15 16:21:20 2010
> New Revision: 121904
>
> URL: http://llvm.org/viewvc/llvm-project?rev=121904&view=rev
> Log:
> Fix opt -Werror build
>
> Modified:
>    llvm/trunk/lib/MC/MCStreamer.cpp
>
> Modified: llvm/trunk/lib/MC/MCStreamer.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/MCStreamer.cpp?rev=121904&r1=121903&r2=121904&view=diff
> ==============================================================================
> --- llvm/trunk/lib/MC/MCStreamer.cpp (original)
> +++ llvm/trunk/lib/MC/MCStreamer.cpp Wed Dec 15 16:21:20 2010
> @@ -49,6 +49,7 @@
>                               unsigned AddrSpace) {
>   assert(Size <= 8 && "Invalid size");
>   unsigned Bits = 8 * Size;
> +  (void)Bits;
>   assert((isUIntN(Bits, Value) || isIntN(Bits, Value)) && "Invalid size");
>   char buf[8];
>
> Or:
> assert((isUIntN(8 * Size, Value) || isIntN(8 * Size, Value)) && "Invalid
> size");
> Seems less lame.

Sounds good to me, patch on the way.

-Matt