Commit: Fix display of 64-bit CFI and FDE frames

Will Newton will.newton@linaro.org
Fri Oct 4 08:38:00 GMT 2013


On 4 October 2013 09:27, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote:
> Hi Will,
>
>
> On 04/10/13 09:05, Will Newton wrote:
>>
>> On 17 September 2013 10:46, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote:
>>
>> Hi Kyrill,
>>
>>>> I'd prefer the regexps to be more consistently written.  Sometimes we
>>>> have <c>*<c><c>123 and sometimes <c>+<c>123
>>>>
>>>> R.
>>>
>>> Ok, I've reworked the patch slightly to use the <c>+<c>123 format
>>> consistently.
>>> Nick already approved the previous version and I think this is an obvious
>>> change
>>> from that one, but just in case I'll wait 24 hours before applying it in
>>> case
>>> anyone has some other comments.
>>
>> This patch should also be applied to the 2.24 branch, as the test is
>> failing.
>
>
> Applied to the binutils-2_24-branch branch. My cvs-foo is rusty, can you
> please check to make sure it's applied correctly?

Looks good to me, no test failures on the branch for aarch64. Thanks!

-- 
Will Newton
Toolchain Working Group, Linaro



More information about the Binutils mailing list