teemperor accepted this revision. teemperor added a comment. This revision is now accepted and ready to land.
To clarify the description a bit: `RegInfoBasedABI::GetRegisterInfoByName` is comparing the name/alt_name by doing a `const char *` pointer value comparison. That only works if both strings that are compared are coming from a ConstString. Since b0060c3a7868ef026d95d0cf8a076791ef74f474 <https://reviews.llvm.org/rGb0060c3a7868ef026d95d0cf8a076791ef74f474> `GetRegisterInfoByName` is checking with an assert that both C strings came from a ConstString (which is failing as these ABI implementation changed here are lacking the ConstString'ification code). Obviously copying this code around isn't ideal, so this patch is just to get the backend at least running again until I get around to refactor this code. Anyway, this LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88375/new/ https://reviews.llvm.org/D88375 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
