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

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

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


https://reviews.llvm.org/D41245