[PATCH] D22758: [compiler-rt] Fix warnings in interception code

Etienne Bergeron via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 25 11:46:14 PDT 2016


etienneb added a comment.

In https://reviews.llvm.org/D22758#495090, @bruno wrote:

> The tests also pass fine here as well. I still see 3 warnings:
>
>   warning: address of function 'memchr' will always evaluate to 'true' [-Wpointer-bool-conversion]
>   
>
> But are you not intending to fix those in this patch, are you?


Yes. It supposed to be fixed.

I think this is the mistake:

  #ifdef SANITIZER_WINDOWS   -->  #if SANITIZER_WINDOWS 




https://reviews.llvm.org/D22758





More information about the llvm-commits mailing list