DavidSpickett added inline comments.

================
Comment at: lldb/source/Plugins/ABI/AArch64/ABIAArch64.cpp:63
+  // should be removed once full PAC support is added.
+  return pc & 0x0000000FFFFFFFFF;
+}
----------------
omjavaid wrote:
> How did you come up with this 36bit mask for PC reg I think this mask is not 
> appropriate for Linux virtual address space which is 52 or 48 bits in length.
> 
> 
If it's any help I looked for this too and found 
`aarch64/functions/pac/calcbottompacbit/CalculateBottomPACBit` pseudocode in 
the armarm. This is used by the PAC strip instructions. (though it is a bit 
mind bending to read)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98529

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

Reply via email to