DavidSpickett marked an inline comment as done. DavidSpickett added inline comments.
================ 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 ---------------- omjavaid wrote: > Should we test for SSVE first that way we wont have to check for SVE once > SSVE is found? Good idea, but according to the architecture supplement: ``` If SME is implemented, this does not imply that FEAT_SVE and FEAT_SVE2 are implemented by the PE when it is not in Streaming SVE mode. ``` I don't know how realistic such a core is but at a glance Linux doesn't say it wouldn't support it. I'll leave this as it is on that basis. 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