<div dir="ltr">I am not quite understand. Existing code work with VS2013. Could please show some examples of the code which is needed, work with your version and do not work with mine.<div>BTW. Also copying should not affect performance as operations on <span style="color:rgb(33,33,33);font-family:"helvetica neue",helvetica,arial,sans-serif;line-height:1.5">AddressDescription </span><span style="line-height:1.5">should be quite rare.</span></div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Sep 8, 2016 at 11:59 AM Filipe Cabecinhas <<a href="mailto:filcab%2Bllvm.phabricator@gmail.com">filcab+llvm.phabricator@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">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 class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Thank you,</div><div class="gmail_msg"> Filipe</div><div class="gmail_msg"><br class="gmail_msg"><div class="gmail_msg"><div class="gmail_msg"><br class="gmail_msg">On Thursday, 8 September 2016, Vitaly Buka <<a href="mailto:vitalybuka@google.com" class="gmail_msg" target="_blank">vitalybuka@google.com</a>> wrote:<br class="gmail_msg"><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">vitalybuka requested changes to this revision.<br class="gmail_msg">
This revision now requires changes to proceed.<br class="gmail_msg">
<br class="gmail_msg">
================<br class="gmail_msg">
Comment at: lib/asan/asan_descriptions.h:185<br class="gmail_msg">
@@ +184,3 @@<br class="gmail_msg">
+  // We should be able to do without this struct after we deprecate VS2013.<br class="gmail_msg">
+  AddressKind kind;<br class="gmail_msg">
+  union {<br class="gmail_msg">
----------------<br class="gmail_msg">
I like this way:  <a href="https://reviews.llvm.org/D24358" class="gmail_msg" target="_blank">https://reviews.llvm.org/D24358</a><br class="gmail_msg">
WDYT?<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D24131" class="gmail_msg" target="_blank">https://reviews.llvm.org/D24131</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div></div></div></blockquote></div>