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 04/10/13 09:27, Kyrill Tkachov 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?
Oops, forgot to wait for Tristans' Ok to backport the patch...
Tristan, is this testsuite patch ok for the 2.24 branch? If you object I can revert it.

Kyrill



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