[Lldb-commits] [PATCH] D100581: [Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable

Michael Benfield via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Jun 2 15:31:56 PDT 2021


mbenfield added a comment.

In D100581#2792854 <https://reviews.llvm.org/D100581#2792854>, @Abpostelnicu wrote:

> I think there is a false positive with this @george.burgess.iv:
> In this <https://searchfox.org/mozilla-central/source/mozglue/baseprofiler/core/platform-linux-android.cpp#222-227> we have the warning triggered: 
> mozglue/baseprofiler/core/platform-linux-android.cpp:216:9: error: variable 'r' set but not used [-Werror,-Wunused-but-set-variable]
>
>   SigHandlerCoordinator() {
>     PodZero(&mUContext);
>     int r = sem_init(&mMessage2, /* pshared */ 0, 0);
>     r |= sem_init(&mMessage3, /* pshared */ 0, 0);
>     r |= sem_init(&mMessage4, /* pshared */ 0, 0);
>     MOZ_ASSERT(r == 0);
>   }

Indeed as pointed out above I think this is a true positive unfortunately.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100581/new/

https://reviews.llvm.org/D100581



More information about the lldb-commits mailing list