Yes, they were already there. I'm making sure they still work. Later we can extend them to work for all errors (or maybe have a better API for error information).<div><br></div><div>Thank you,</div><div> Filipe<span></span><br><br>On Tuesday, 20 September 2016, Vitaly Buka <<a href="mailto:vitalybuka@google.com">vitalybuka@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">vitalybuka accepted this revision.<br>
vitalybuka added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM<br>
BTW. arc patch <a href="https://reviews.llvm.org/D24555" target="_blank">https://reviews.llvm.org/<wbr>D24555</a> does not work.<br>
<br>
<br>
================<br>
Comment at: lib/asan/asan_descriptions.h:<wbr>230<br>
@@ -230,1 +229,3 @@<br>
+        // initialization-order-fiasco has a special Print()<br>
+        return data.global.Print(bug_descr);<br>
     }<br>
----------------<br>
Maybe better to extract special address kind for odr?<br>
<br>
================<br>
Comment at: lib/asan/asan_report.cc:430<br>
@@ -638,3 +429,3 @@<br>
 int __asan_report_present() {<br>
-  return report_happened ? 1 : 0;<br>
+  return ScopedInErrorReport::<wbr>CurrentError().kind == kErrorKindGeneric;<br>
 }<br>
----------------<br>
Why all these only for Generic?<br>
Is this pre-existed behavior?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D24555" target="_blank">https://reviews.llvm.org/<wbr>D24555</a><br>
<br>
<br>
<br>
</blockquote></div>