clayborg added a comment. Looks fine to me. A few nits.
================ Comment at: lldb/include/lldb/Utility/RegularExpression.h:98 + /// The compiled regular expression. + mutable llvm::Regex m_regex; }; ---------------- Why does this need to be mutable? ================ Comment at: lldb/source/Commands/CommandObjectFrame.cpp:1 -//===-- CommandObjectFrame.cpp ----------------------------------*- C++ -*-===// +//===-- CommandObrectFrame.cpp ----------------------------------*- C++ -*-===// // ---------------- revert CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66174/new/ https://reviews.llvm.org/D66174 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
