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


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?

Thanks,
Kyrill


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