DavidSpickett added a comment. In D99947#2719403 <https://reviews.llvm.org/D99947#2719403>, @omjavaid wrote:
> In D99947#2718954 <https://reviews.llvm.org/D99947#2718954>, @DavidSpickett > wrote: > >> Did the changes in `GDBRemoteRegisterContext.cpp` >> (https://reviews.llvm.org/D99947?id=335484#inline-940747 ) get moved to >> another diff? > > We no longer need to configure Address bits in use after changes from D100521 > <https://reviews.llvm.org/D100521> and D99944 > <https://reviews.llvm.org/D99944> get merged. ABISysV_arm64 now contains the > logic for masking of ignored bits from code and data addresses. Look through > overrides of FixCodeAddress and FixDataAddress in D99944 > <https://reviews.llvm.org/D99944>. Seems like you could merge them completely in that case. Make one commit that adds tests for core files and real targets. (FWIW this LGTM with the comment for _start added) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99947/new/ https://reviews.llvm.org/D99947 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits