Erase NDBEUG from all Release targets

Aaron Ballman aaron at aaronballman.com
Thu Sep 19 09:14:40 PDT 2013


LGTM, but I defer to cmake wizards for final sign-off.

~Aaron

On Thu, Sep 19, 2013 at 11: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
>



More information about the llvm-commits mailing list