labath added a comment.

In D69371#1721077 <https://reviews.llvm.org/D69371#1721077>, @omjavaid wrote:

> We ll be dealing with Linux user mode and mostly aarch64 data registers 
> except for cpsr, fpsr and fpcr. I think we should be fine but let me confirm 
> this again from documentation.


Right, but you're creating a general interface for all architectures, not just 
several aarch64 registers. Even if they don't make use of that facility now, it 
would be good to make sure they can do that in the future.

For instance, on x86, the kernel may decide to reject 
https://github.com/torvalds/linux/blob/master/arch/x86/kernel/ptrace.c#L187 
some values of some registers, and silently ignore some bits in others 
https://github.com/torvalds/linux/blob/master/arch/x86/kernel/ptrace.c#L349. 
That's why I think it would be better to commit changes to memory 
automatically/immediately, and minimize the chances that subsequent "read" 
operations will return data which does not reflect the actual values held by 
the OS.


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

https://reviews.llvm.org/D69371



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to