public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [AArch64/gas] v[0-31] registers not allowed in cfi_offset?
@ 2014-08-06 14:07 Steven Vormwald
  2014-08-06 22:48 ` Richard Henderson
  0 siblings, 1 reply; 2+ messages in thread
From: Steven Vormwald @ 2014-08-06 14:07 UTC (permalink / raw)
  To: binutils

Are the v[0-31] registers intentionally not supported in cfi_offset?
I'm trying to figure out whether this is an oversight in gas due to
aarch64 support being relatively new (seems likely given bug 16694) or
if they are intentionally not supported and I need to stop emitting
cfi_offset with them.

Thanks,

Steven Vormwald

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [AArch64/gas] v[0-31] registers not allowed in cfi_offset?
  2014-08-06 14:07 [AArch64/gas] v[0-31] registers not allowed in cfi_offset? Steven Vormwald
@ 2014-08-06 22:48 ` Richard Henderson
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Henderson @ 2014-08-06 22:48 UTC (permalink / raw)
  To: Steven Vormwald, binutils

On 08/06/2014 04:06 AM, Steven Vormwald wrote:
> Are the v[0-31] registers intentionally not supported in cfi_offset?
> I'm trying to figure out whether this is an oversight in gas due to
> aarch64 support being relatively new (seems likely given bug 16694) or
> if they are intentionally not supported and I need to stop emitting
> cfi_offset with them.

You mean by name?  Technically, only the low 64-bits is supportable in the
unwind info, i.e. d[0-31].  So I think we shouldn't allow the V registers by
name.  Or under the B, H or S names either, come to that.

But I also see that we produce invalid unwind info for the D regs:


	.cfi_startproc
foo:
	nop
	.cfi_offset x10,0
	.cfi_offset d10,0
	.cfi_endproc

00000014 00000018 00000018 FDE cie=00000000 pc=00000000..00000004
  DW_CFA_advance_loc: 4 to 00000004
  DW_CFA_offset: r10 at cfa+0
  DW_CFA_offset: r10 at cfa+0

That second opcode should be r42.


r~

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-08-06 22:48 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-06 14:07 [AArch64/gas] v[0-31] registers not allowed in cfi_offset? Steven Vormwald
2014-08-06 22:48 ` Richard Henderson

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).