jasonmolenda added inline comments.

================
Comment at: lldb/source/Plugins/Process/mach-core/ThreadMachCore.cpp:112
+          auto exception_class =
+              static_cast<AppleArm64ExceptionClass>(esr_val >> 26);
+          if (exception_class !=
----------------
shafik wrote:
> Does `26` have a meaning? I am guessing we are shifting to get the EC bits?
Yeah I think the comment table in AppleArm64ExceptionClass.def would be better 
placed here, showing the bit positions of the three fields in the esr register 
probably.  AppleArm64ExceptionClass.def should make it clear that it is 
operating on the exception class field of the esr register, but the bit 
positions are more relevant here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109795

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

Reply via email to