<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Nov 19, 2013, at 11:52 AM, Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="rtl"><div dir="ltr">Hi Chris,</div><div dir="ltr"><br></div><div dir="ltr">No, I don't have such system. </div><div dir="ltr"><br></div><div dir="ltr">While reading the code for another purpose I noted the discrepancy between the comment saying LineCol is 32 bit, the type itself and the get column method which will completely fail with non 32-bit unsigned.</div>
<div dir="ltr"><br></div><div dir="ltr">I'm sure there are more problems - one less to go!</div></div></blockquote><div><br></div><div>Ok, but what I’m saying is: please do not file bugs or change the code to make this happen. Fixing problems for theoretical architectures is not a good use of time, and code churn for no reason is bad.</div><div><br></div><div>-Chris</div><div><br></div><br><blockquote type="cite"><div dir="rtl"><div dir="ltr"><br></div><div dir="ltr">Yaron<br></div><div dir="ltr"><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
<div dir="ltr">2013/11/19 Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com" target="_blank">clattner@apple.com</a>></span></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
On Nov 17, 2013, at 1:47 AM, Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>> wrote:<br>
<br>
> Author: yrnkrn<br>
> Date: Sun Nov 17 03:47:39 2013<br>
> New Revision: 194957<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=194957&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=194957&view=rev</a><br>
> Log:<br>
> DebugLoc defines LineCol as 32 bit in comment but unsigned in code.<br>
> This patch modifies LineCol to be a uint32_t.<br>
><br>
> See <a href="http://llvm.org/bugs/show_bug.cgi?id=17957" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=17957</a><br>
<br>
</div>Are you really trying to build LLVM on a system where “unsigned” is 64-bits? You’re going to run into pervasive problems, and it isn’t clear to me that fixing them piecemeal is going to work. LLVM and Clang generally assume that int and unsigned are both 32-bits.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Chris</font></span></blockquote></div><br></div>
</blockquote></div><br></body></html>