<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 6, 2015 at 2:31 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@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">As long as it doesn't bloat size let's go for it. The original limitation was so that column and line would fit into a single 32-bits.</div></blockquote><div><br></div><div>IIRC clang's SourceLocation is only 32 bits anyway, so theoretically that should be all we need (I assume other language frontends will have less extreme requirements w.r.t. source locations than clang).</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class="HOEnZb"><font color="#888888"><br><br>-eric<br></font></span></div><div class="HOEnZb"><div class="h5"><br><div class="gmail_quote">On Fri Feb 06 2015 at 2:26:22 PM Duncan P. N. Exon Smith <<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Remove unnecessary restriction of 32-bits for line numbers in<br>
`MDLocation`.<br>
<br>
The rest of the debug info schema (with the exception of local<br>
variables) uses 32-bits for line numbers.  As I introduce the<br>
specialized nodes, it makes sense to canonicalize on one size or the<br>
other.<br>
<br>
Does this make sense to you folks, or would you rather canonicalize<br>
on 24-bits?<br>
<br>
</blockquote></div>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div></div>