This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]