[PATCH] D31702: Append -w when LLVM_ENABLE_WARNINGS is Off.

Vassil Vassilev via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 4 03:31:15 PDT 2020


On 6/4/20 6:58 AM, Sean Silva wrote:
> Can we revert this?
>
> In our downstream project we were bitten by this: 
> https://github.com/google/mlir-npcomp/commit/cd7258dbd48bd9254c6741cab2d3f4e36cbd3c84
>
> It seems that now by default anybody that depends on LLVM with CMake 
> will globally get a `-w` passed to their compiler. That doesn't seem 
> like good behavior.


   Wouldn't that be the case for all append clauses in that file?


>
> -- Sean Silva
>
> On Wed, Apr 12, 2017 at 1:56 PM Vassil Vassilev via Phabricator via 
> llvm-commits <llvm-commits at lists.llvm.org 
> <mailto:llvm-commits at lists.llvm.org>> wrote:
>
>     v.g.vassilev closed this revision.
>     v.g.vassilev added a comment.
>
>     Landed in r300100.
>
>
>     https://reviews.llvm.org/D31702
>
>
>
>     _______________________________________________
>     llvm-commits mailing list
>     llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>
>     http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200604/dda42bc6/attachment.html>


More information about the cfe-commits mailing list