<div dir="ltr">I'm happy to have either representation, but I would like it to be consistent between pointers and references. One point that speaks for not having it in the debug info is that it makes the debug info dependent on the size of the pointer (when fixing up the test cases I had to add quite a number of explicit `target triple`s to fix the pointer size).<div><br></div><div>However, is it possible that different pointer types (e.g. in different address spaces) could have different size? In that case it might be needed in either metadata or DWARF or both.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 3, 2015 at 1:13 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Mon, Nov 2, 2015 at 10:04 PM, Keno Fischer via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">loladiro added a comment.<br>
<br>
Ah, interesting we skip emitting the size of a pointer in the backend. Will add the same for reference types.<br></blockquote><div><br></div></span><div>Oh, interesting...</div><div><br></div><div>Suspicious why we add it to the metadata then... that seems strange. (admittedly it comes in handy for the verification you're adding - but I'm not sure how much we want to add to the metadata to facilitate that sort of thing & make the debug info less clear in its purpose by doing so)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
<span><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D14275" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14275</a><br>
<br>
<br>
<br>
</span></span><span class=""><div><div>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></span></blockquote></div><br></div></div>
</blockquote></div><br></div>