omjavaid accepted this revision. omjavaid added a comment. This revision is now accepted and ready to land.
This looks very good much simpler from the first version. just a minor nit. ================ Comment at: lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_arm64.cpp:81 opt_regsets.Set(RegisterInfoPOSIX_arm64::eRegsetMaskSVE); + // We may also have the Scalable Matrix Extension (SME) which adds a ---------------- Should we test for SSVE first that way we wont have to check for SVE once SSVE is found? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154926/new/ https://reviews.llvm.org/D154926 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits