DavidSpickett added a comment. What I understand is that `MCInstrAnalysis` defaults to using `MCInstrDesc`, however not all targets have `MCInstrAnalysis` at all. So we have this awkwardness here with the two pointers, which is fine.
Sounds good to me, I am just wondering about the changes over in llvm. ================ Comment at: llvm/include/llvm/MC/MCInstrAnalysis.h:80 + } + /// Returns true if at least one of the register writes performed by ---------------- Does this have test coverage already? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156086/new/ https://reviews.llvm.org/D156086 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
