<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">LGTM. A test that uncovers could be
useful, since we do have the RoundTrip passes that could be used
to verify the change.<br>
<br>
On 11/4/2013 12:05 PM, Simon Atanasyan wrote:<br>
</div>
<blockquote
cite="mid:CAGyS+DSLan-=ri6m=6VztG3HCawjepDn_SJPN9eEYMH5Tm7xhA@mail.gmail.com"
type="cite">
<pre wrap="">Hi,
It looks like the File::processTargetsTable() routine has a typo in
the expression calculates an absolute atom offset. It uses slIndex
instead of abIndex. But I am not an expert in lld code so I am not
sure for 100%.
Could you please to take a look at the patch. If it is OK I will
commit the patch.
Regards,
Simon
</pre>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
llvm-commits mailing list
<a class="moz-txt-link-abbreviated" href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>
<a class="moz-txt-link-freetext" href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a>
</pre>
</blockquote>
<br>
<br>
<pre class="moz-signature" cols="72">--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation</pre>
</body>
</html>