[PATCH] D38515: Fix ISO C++ fnptr -> objptr errors.

Mitch Phillips via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 3 15:51:30 PDT 2017


hctim added a comment.

In https://reviews.llvm.org/D38515#887573, @pcc wrote:

> What's the motivation for this patch? Are you trying to make LLVM compile with `-Wpedantic`? Is this the only issue, or are there others?


Currently, if you build compiler-rt as part of llvm, `-DLLVM_ENABLE_PEDANTIC` is default-set to true. This results in warnings being shown on the default compile.

There are currently 17 warnings produced by compiling llvm with default parameters. compiler-rt is responsible for all 17. Most of them are from `-Wall` or `-Wextra` and I'm working through them sequentially. This patch fixes 6/17 warnings, making `-Wpedantic` no longer cause errors.


https://reviews.llvm.org/D38515





More information about the llvm-commits mailing list