[Lldb-commits] [PATCH] D91504: [lldb] Use translated full ftag values [WIP]

[Michał Górny via Phabricator via lldb-commits]

mgorny added inline comments.


================
Comment at: lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.cpp:534
+  // TODO
+  if (reg_info->kinds[lldb::eRegisterKindLLDB] == lldb_ftag_x86_64) {
+    uint8_t abridged_tw = *(uint8_t *)src;
----------------
labath wrote:
> mgorny wrote:
> > @labath, any suggestion what kind of test to use here? Maybe against `&m_xstate->fxsave.ftag` address?
> You mean, like, how to detect that we're dealing with the ftag register?
> 
> This approach seems reasonable to me..
This = checking register number like in the code (in which I suppose we'd have to move distinguishing amd64/i386 to ctor like `first_fpr`? Or checking for actual pointer as I suggested in the comment?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91504/new/

https://reviews.llvm.org/D91504