<div dir="rtl"><div dir="ltr">Hi Reid,</div><div dir="ltr"><br></div><div dir="ltr">I don't know how (gmail tabs?) but I never saw your reply, sorry for the late response.</div><div dir="ltr"><br></div><div dir="ltr">I'm attaching the same patch against a newer r192756,  please submit this, I don't have commit rights.</div>

<div dir="ltr"><br></div><div dir="ltr">Yaron</div><div dir="ltr"><br></div><div dir="ltr"><br></div></div><div class="gmail_extra"><div dir="ltr"><br><br><div class="gmail_quote">2013/9/19 Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span><br>

<blockquote class="gmail_quote" style="margin:0 .8ex;border-left:1px #ccc solid;border-right:1px #ccc solid;padding-left:1ex;padding-right:1ex">Yeah, go for it.  Do you need someone to commit it?</blockquote></div><div class="gmail_extra">

<br><br><div class="gmail_quote"><div><div class="h5">On Thu, Sep 19, 2013 at 8:50 AM, Yaron Keren <span dir="ltr"><<a href="mailto:yaron.keren@gmail.com" target="_blank">yaron.keren@gmail.com</a>></span> wrote:<br>


</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="rtl"><div dir="ltr" style="font-family:arial,sans-serif;font-size:12.499999046325684px">

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.</div>




<div dir="ltr" style="font-family:arial,sans-serif;font-size:12.499999046325684px"><br></div><div dir="ltr" style="font-family:arial,sans-serif;font-size:12.499999046325684px">The attached patch addresses the issue.</div>


<span><font color="#888888">

<div dir="ltr" style="font-family:arial,sans-serif;font-size:12.499999046325684px"><br></div><div dir="ltr" style="font-family:arial,sans-serif;font-size:12.499999046325684px">Yaron</div><div dir="ltr" style="font-family:arial,sans-serif;font-size:12.499999046325684px">




<br></div><div style="font-family:arial,sans-serif;font-size:12.499999046325684px"></div></font></span></div>
<br></div></div><div class="im">_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></div></blockquote></div><br></div></div></div>