[PATCH] Handle set but unused warning on Release No Assertions builds

Justin Bogner mail at justinbogner.com
Wed Mar 11 23:54:35 PDT 2015


"Edwards, Michael" <Michael_Edwards at playstation.sony.com> writes:
> Our bots are generating warnings on some of our Release builds because we have
> Assertions turned off.  Attached is a patch which should help to alleviate
> this problem.  If someone could review this and if it is acceptable please
> commit it for me.  I do not have commit privileges.  Thanks in advance.

Sure, this is obviously correct. r232015.

> Respectfully,
> Mike Edwards
>
> Mike Edwards
>
> Staff DevOps Engineer
> Platform Architecture
> Sony PlayStation® Worldwide Studios
>
> 2207 Bridgepointe Pkwy.
> Mailstop 23163
> San Mateo, CA 94404
>
> mike_edwards at playstation.sony.com
> P:  650.655.5352
> C:  650.483.0328
> IRC: sqlbyme
>
>
> diff --git a/lib/ReaderWriter/MachO/StubsPass.cpp b/lib/ReaderWriter/MachO/StubsPass.cpp
> index 0e97f35..e320ea5 100644
> --- a/lib/ReaderWriter/MachO/StubsPass.cpp
> +++ b/lib/ReaderWriter/MachO/StubsPass.cpp
> @@ -274,6 +274,7 @@ public:
>        }
>      }
>      assert(binderFound && "dyld_stub_binder not found");
> +    (void)binderFound;
>  
>      // Sort targets by name, so stubs and lazy pointers are consistent
>      std::vector<const Atom *> targetsNeedingStubs;
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list