Re: [PATCH v2 0/3] Small step in supporting AVX instructions

[Guinevere Larsen <blarsen@redhat.com>]

On 6/25/24 4:28 AM, Willgerodt, Felix wrote:
>> -----Original Message-----
>> From: Guinevere Larsen <blarsen@redhat.com>
>> Sent: Montag, 24. Juni 2024 19:49
>> To: Guinevere Larsen <blarsen@redhat.com>; gdb-patches@sourceware.org
>> Subject: Re: [PATCH v2 0/3] Small step in supporting AVX instructions
>>
>> I've made the mistake of wondering about YMM, and noticed that this
>> series fails to save the higher bits of the register.
>>
>> Seeing as it isn't possible to save ymm registers directly yet (more
>> info here[1][2]), and I'd like to avoid having to save ymmh and xmm in
>> this series, I'll wait until the relevant series get approved before
>> sending a v3 of this series
>>
>> --
>> Cheers,
>> Guinevere Larsen
>> She/Her/Hers
>>
> Hi Guinevere,
>
> Did you forget to include links for [1] and [2]? I can't find what you are
> Referencing in your email. And also can't remember reading anything
> related on the mailing list.

oops, yes i did.

[1] 
https://inbox.sourceware.org/gdb-patches/20240621180730.859487-1-blarsen@redhat.com/
[2] 
https://inbox.sourceware.org/gdb-patches/20240621180918.860097-2-blarsen@redhat.com/

>
> And yes the SSE/AVX registers have seen a bit of historical growth, and are
> split up over different XSTATE regions. Though I don't see a way around this
> split in GDB.

Those 5 patches combined should make it possible to read 
pseudo-registers from the regcache we save for the record target.

I say should because I'm still getting some oddities when rebasing my 
series on top of them and testing the changes, though, so I might be 
missing something. I also don't really know anything about XSTATE, so if 
that's also a problem, do let me know!

We could also just save xmm and ymmh registers, but I think using ymm is 
more readable, so I'd prefer to use that if possible.

-- 
Cheers,
Guinevere Larsen
She/Her/Hers

>
> Felix
> Intel Deutschland GmbH
> Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
> Tel: +49 89 99 8853-0, www.intel.de
> Managing Directors: Sean Fennelly, Jeffrey Schneiderman, Tiffany Doon Silva
> Chairperson of the Supervisory Board: Nicole Lau
> Registered Office: Munich
> Commercial Register: Amtsgericht Muenchen HRB 186928