[PATCH] Add extended mnemonics for bctar. Fix setting of 'at' branch hints.
Peter Bergner
bergner@linux.ibm.com
Thu Apr 4 14:03:00 GMT 2019
On 4/4/19 7:49 AM, Alan Modra wrote:
> On Wed, Apr 03, 2019 at 03:54:56PM -0500, Peter Bergner wrote:
>> Anton complained that the bctar instruction was missing extended mnemonics
>> to make reading disassembly more readable. In the process of adding that
>> support, I noticed that we were not correctly setting 'at' branch hints
>> for many branches. Instead, we were incorrectly setting the 1-bit 'y'
>> branch hint, which wasn't correct on anything POWER4 and later.
>>
>> The patch below adds the missing bctar extended mnemonics as well as
>> now correctly setting and extracting 'at' branch hints. The patch
>> flagged some illegal tests within the a2.[ds] tests, so I fixed those
>> to use legal operand values.
>>
>> This passed testing with no errors. I plan on committing this tomorrow.
>
> So I wrote another testcase to check all the possible BO bits with
> bc, bcctr, bclr and bctar, and noticed we have some problems remaining
> with disassembly of insns using 'y' hints. I'll commit the testcase
> and fixes after your patch goes in.
Ok, I committed my patch. I'll be interested in seeing the other
issues you've discovered.
Peter
More information about the Binutils
mailing list