<div dir="ltr">Just FYI;<div>If you want to get this metadata into trunk, you are going to have to have a real design RFC for it, and, as part of that, explain why you can't or shouldn't generalize something existing (like the struct path metadata etc), to this case.</div><div><br></div><div>Not suggesting you can't make that argument, of course, just making sure you realize you'll have to actually be able to defend new metadata, we don't just add everything :)</div><div><br></div><div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 22, 2016 at 2:55 PM, Taewook Oh <span dir="ltr"><<a href="mailto:twoh@fb.com" target="_blank">twoh@fb.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">twoh added a comment.<br>
<br>
@dberlin I was planning to add metadata only for structures (not unions), but even in that case "offset and size" seems like a better idea. Thanks!<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D20665" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20665</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>