[lld][PATCH] Fix typo in the File::processTargetsTable() routine

Shankar Easwaran shankare at codeaurora.org
Mon Nov 4 10:42:18 PST 2013


LGTM. A test that uncovers could be useful, since we do have the 
RoundTrip passes that could be used to verify the change.

On 11/4/2013 12:05 PM, Simon Atanasyan wrote:
> 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
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits


-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131104/b0f9ae85/attachment.html>


More information about the llvm-commits mailing list