<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 5, 2015 at 2:36 PM, Alexey Samsonov <span dir="ltr"><<a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">samsonov added a comment.<br>
<br>
The correct header for this is `lib/sanitizer_common/sanitizer_internal_defs.h`.</blockquote><div><br>Done<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> BTW we already have `NOTHROW` define there, but it does smth. different.<br></blockquote><div><br>I updated the side of NOTHROW that defines it as "throw()" to use the NOEXCEPT macro (so it'll use the non-deprecated noexcept, where available). <br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
================<br>
Comment at: lib/tsan/rtl/tsan_new_delete.cc:39<br>
@@ +38,3 @@<br>
+#else<br>
+#define LLVM_NOEXCEPT LLVM_NOEXCEPT<br>
+#endif<br>
----------------<br>
Huh?<br></blockquote><div><br></div><div>Ah, probably did that accidentally when I s/throw()/NOEXCEPT/ in the file to try to update all the existing sites.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="http://reviews.llvm.org/D11780" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11780</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>