If I could stop caring about VS2013, that would be good enough. My problem is that having it this way will force us to copy the data whenever we want to use it from a structure that need to be put in a union (because of VS2013), since you can only put a *Data in there, not the full AddressDescription.<div><br></div><div>Thank you,</div><div> Filipe<br><div><div><br>On Thursday, 8 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 requested changes to this revision.<br>
This revision now requires changes to proceed.<br>
<br>
================<br>
Comment at: lib/asan/asan_descriptions.h:<wbr>185<br>
@@ +184,3 @@<br>
+  // We should be able to do without this struct after we deprecate VS2013.<br>
+  AddressKind kind;<br>
+  union {<br>
----------------<br>
I like this way:  <a href="https://reviews.llvm.org/D24358" target="_blank">https://reviews.llvm.org/<wbr>D24358</a><br>
WDYT?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D24131" target="_blank">https://reviews.llvm.org/<wbr>D24131</a><br>
<br>
<br>
<br>
</blockquote></div></div></div>