[Lldb-commits] [PATCH] D41245: Reduce x86 register context boilerplate.

[Kamil Rytarowski via Phabricator via lldb-commits]

krytarowski added a comment.

In https://reviews.llvm.org/D41245#956644, @labath wrote:

> In https://reviews.llvm.org/D41245#955557, @krytarowski wrote:
>
> > Maybe reuse FPR for FXSAVE/FSAVE and add next to it XSAVE/XSAVE_OPT.
>
>
> I am not sure what you mean by that, but I don't think it can work that way, as xsave area already contains a copy of the fpu registers, so we will end up having them twice. I have a feeling that for supporting fsave you will need to define your own register context (or just handle the transformation at a different layer -- this is how we avoided defining separate register context for minidumps -- we just rearrange the registers around to match the existing register context layout).


I was thinking about split of FPU into two or more structs/unions.

But the newer version on review is better!


https://reviews.llvm.org/D41245