Erase NDBEUG from all Release targets
aaron at aaronballman.com
Thu Sep 19 09:14:40 PDT 2013
LGTM, but I defer to cmake wizards for final sign-off.
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.
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
More information about the llvm-commits