public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Alan Modra <amodra@gmail.com>
To: Tejas Belagod <Tejas.Belagod@arm.com>
Cc: "binutils@sourceware.org" <binutils@sourceware.org>,
	Richard Earnshaw <Richard.Earnshaw@arm.com>
Subject: Re: [Patch, Arm, Readelf] Support for a new pacbti unwind opcode.
Date: Sun, 31 Oct 2021 13:20:37 +1030	[thread overview]
Message-ID: <YX4EfSnV517fm3gC@squeak.grove.modra.org> (raw)
In-Reply-To: <AS8PR08MB7079B46A12E830405DBCDBA5EA879@AS8PR08MB7079.eurprd08.prod.outlook.com>

On Fri, Oct 29, 2021 at 02:50:02PM +0000, Tejas Belagod wrote:
> 
> 
> > -----Original Message-----
> > From: Alan Modra <amodra@gmail.com>
> > Sent: Friday, October 29, 2021 11:09 AM
> > To: Tejas Belagod <Tejas.Belagod@arm.com>
> > Cc: binutils@sourceware.org; Richard Earnshaw
> > <Richard.Earnshaw@arm.com>
> > Subject: Re: [Patch, Arm, Readelf] Support for a new pacbti unwind opcode.
> > 
> > On Thu, Oct 28, 2021 at 02:41:57PM +0000, Tejas Belagod via Binutils wrote:
> > > Ping
> > >
> > 
> > Applied.  Would you mind having a look at this fail?
> > 
> > armeb-linuxeabi  +FAIL: Unwind information for Armv8.1-M.Mainline PACBTI
> > extension
> > 
> 
> gas/testsuite/arm/unwind-pacbti-m.d only tests for Little-Endian targets.
> Disable it for Big-Endian targets.

Is there a reason the test should not run for BE?  How about this
instead?

	* testsuite/gas/arm/unwind-pacbti-m.d: Match armeb output.

diff --git a/gas/testsuite/gas/arm/unwind-pacbti-m.d b/gas/testsuite/gas/arm/unwind-pacbti-m.d
index 584b12037b3..28021758a74 100644
--- a/gas/testsuite/gas/arm/unwind-pacbti-m.d
+++ b/gas/testsuite/gas/arm/unwind-pacbti-m.d
@@ -14,10 +14,10 @@ OFFSET   TYPE              VALUE
 
 
 Contents of section .text:
- 0000 10b54df8 04cd5df8 04cb10bd  .*
+ 0000 (10b54df8 04cd5df8 04cb10bd|b510f84d cd04f85d cb04bd10)  .*
 Contents of section .ARM.exidx:
- 0000 00000000 b0a8b480  .*
+ 0000 00000000 (b0a8b480|80b4a8b0)  .*
 Contents of section .ARM.attributes:
- 0000 41290000 00616561 62690001 1f000000  .*
+ 0000 41(290000 00|000000 29)616561 62690001 (1f000000|0000001f)  .*
  0010 05382e31 2d4d2e4d 41494e00 0615074d  .*
  0020 09033202 34024a01 4c01               .*


-- 
Alan Modra
Australia Development Lab, IBM

  reply	other threads:[~2021-10-31  2:50 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-05  8:43 Tejas Belagod
2021-10-28 14:41 ` Tejas Belagod
2021-10-29 10:09   ` Alan Modra
2021-10-29 12:45     ` Tejas Belagod
2021-10-29 14:50     ` Tejas Belagod
2021-10-31  2:50       ` Alan Modra [this message]
2021-11-01 12:05         ` Tejas Belagod
2021-11-02 12:13         ` Tejas Belagod

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YX4EfSnV517fm3gC@squeak.grove.modra.org \
    --to=amodra@gmail.com \
    --cc=Richard.Earnshaw@arm.com \
    --cc=Tejas.Belagod@arm.com \
    --cc=binutils@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).