<div dir="ltr"><div dir="ltr"><div><br clear="all"></div></div><div>OK.</div><div><br></div><div>The incorrect address calculation happens on 64-bit hosts as 0xffff0000 remains a positive value.<br></div><div><br></div><div>I have not checked 32-bit hosts, but surely sign extension should work as expected there.<br></div><div><br></div><div>gopi</div><div><br></div><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Mon, Jan 6, 2025 at 7:35 PM Michael Eager <<a href="mailto:eager@eagercon.com">eager@eagercon.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Gopi --<br>
<br>
I'm out of the office until Jan 16.  I'll apply the patch when I return.<br>
<br>
I presume that the incorrect address calcluation only happened on 64-bit <br>
hosts.<br>
<br>
On 1/5/25 11:29 PM, Gopi Kumar Bulusu wrote:<br>
> <br>
>     From: Michael Eager <<a href="mailto:eager@eagercon.com" target="_blank">eager@eagercon.com</a> <mailto:<a href="mailto:eager@eagercon.com" target="_blank">eager@eagercon.com</a>>><br>
>     To: <a href="mailto:binutils@sourceware.org" target="_blank">binutils@sourceware.org</a> <mailto:<a href="mailto:binutils@sourceware.org" target="_blank">binutils@sourceware.org</a>><br>
>     Date: Sat, 4 Jan 2025 14:47:43 -0800<br>
>     Subject: Re: [PATCH v1] Microblaze: Widen mask used in opcodes/<br>
>     microblaze-dis.c<br>
>     On 12/31/24 9:26 PM, Gopi Kumar Bulusu wrote:<br>
> <br>
>     Hi Gopi --<br>
> <br>
>     Do you have write access to the Binutils repo?<br>
> <br>
>     If you do, this patch is OK to check in.  Please add a comment<br>
>     in the git log.<br>
> <br>
>     If you don't, I'll apply this when I have an opportunity.<br>
> <br>
> <br>
> Hello Mike,<br>
> <br>
> I don't have write access.<br>
> <br>
> Appreciate your help.<br>
> dhanyavadam<br>
> gopi<br>
> <br>
<br>
-- <br>
Michael Eager<br>
<br>
</blockquote></div></div>