[PATCH][compiler-rt] Fix global constructor warning in sanitizer.

Kostya Serebryany kcc at google.com
Mon Sep 15 13:51:07 PDT 2014


LGTM

On Wed, Sep 10, 2014 at 4:32 PM, Samuel F Antao <sfantao at us.ibm.com> wrote:

> Hi all,
>
> If EV_VERSION > 0x010000 is not true, the compilation of
> sanitizer_platform_limits_posix.cc returns the warnings:
>
> .../projects/compiler-rt/lib/sanitizer_common/sanitizer_platform_limits_posix.cc:757:12:
> warning: declaration requires a global constructor [-Wglobal-constructors]
>   unsigned IOCTL_EVIOCGKEYCODE_V2 = IOCTL_NOT_PRESENT;
>            ^                        ~~~~~~~~~~~~~~~~~
> .../projects/compiler-rt/lib/sanitizer_common/sanitizer_platform_limits_posix.cc:758:12:
> warning: declaration requires a global constructor [-Wglobal-constructors]
>   unsigned IOCTL_EVIOCGPROP = IOCTL_NOT_PRESENT;
>            ^                  ~~~~~~~~~~~~~~~~~
> .../projects/compiler-rt/lib/sanitizer_common/sanitizer_platform_limits_posix.cc:759:12:
> warning: declaration requires a global constructor [-Wglobal-constructors]
>   unsigned IOCTL_EVIOCSKEYCODE_V2 = IOCTL_NOT_PRESENT;
>            ^                        ~~~~~~~~~~~~~~~~~
> 3 warnings generated.
>
> This patch fixes the issue by adding the const attribute to
> IOCTL_NOT_PRESENT.
>
> The modified files are the following:
> sanitizer_platform_limits_posix.cc
> sanitizer_platform_limits_posix.h
>
>
> Thanks,
> Sam
>
> *(See attached file: crt_ioctl_not_present_fix.patch)*
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140915/651b294d/attachment.html>


More information about the llvm-commits mailing list