[Lldb-commits] [PATCH] D48351: Move dumping code out of RegisterValue class

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Jul 20 10:07:55 PDT 2018


jingham added a comment.

Dump is really meant to be the private description of the object that you would use for logging and the like - Description was the public face of a class.  So while the Description-like functionality could have a restrictions that constrain its use to pretty high up in the stack, you really want to be able to Dump some object you've been handed - particularly into the log stream - wherever that might help.  So I don't think moving to restrict where you can use Dump is a good idea.


https://reviews.llvm.org/D48351





More information about the lldb-commits mailing list