Erase NDBEUG from all Release targets

Reid Kleckner rnk at google.com
Thu Sep 19 09:17:57 PDT 2013


Yeah, go for it.  Do you need someone to commit it?


On Thu, Sep 19, 2013 at 8:50 AM, Yaron Keren <yaron.keren at gmail.com> wrote:

> While patching HandleLLVMOptions, I noticed that NDBEUG is removed only
> from the release compile flags but not from the minimum size release or the
> release with debug info.  With these targets Visual C++ warns about NDEBUG
> being defined and undefined.
>
> The attached patch addresses the issue.
>
> Yaron
>
>
> _______________________________________________
> 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/20130919/a74978e5/attachment.html>


More information about the llvm-commits mailing list