On 31 August 2012 22:10, Michael Spencer <span dir="ltr"><<a href="mailto:bigcheesegs@gmail.com" target="_blank">bigcheesegs@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5">On Fri, Aug 31, 2012 at 6:44 PM, Nick Lewycky <<a href="mailto:nlewycky@google.com">nlewycky@google.com</a>> wrote:<br>
> The attached patch updates libObject's code for producing a string<br>
> representing a relocation. This lets us emit "Unknown" far less often. The<br>
> test is a .o assembled from this .s:<br>
><br>
> .text<br>
>   func:<br>
>     movb $somedata-4, %al<br>
>     movw $somedata-4, %ax<br>
>     movl $somedata-4, %eax<br>
>     movq $somedata-4, %rax<br>
>     movabsl somedata-4, %eax<br>
>     movl %eax, somedata(%rip)<br>
><br>
>     movl $somedata, %ebx<br>
><br>
>     movl $somedata+4, %ecx<br>
><br>
> .data<br>
>   somedata:<br>
>     .long 0x0<br>
><br>
> Please review!<br>
><br>
> Nick<br>
><br>
><br>
</div></div>> _______________________________________________<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>
<br>
Is fmt.flush() really needed? Other than that LGTM.<br></blockquote><div><br></div><div>Sadly it is. Thanks for the review!</div><div><br></div><div>Nick</div><div><br></div></div>