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

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

In https://reviews.llvm.org/D48351#1168384, @jingham wrote:

> If this pattern becomes more common, then we have to deal with how somebody would find these dump routines.  If RegisterValue gets moved out of Core, would you really look to a file in Core for the dump routine, especially as many other classes have an obvious Dump method?


Do you think it would help if I merged this new file with the existing `DumpDataExtractor.cpp` into something like `Core/Dumpers.h` (I've considered something like that initially, but then chose not to in order to reduce rename churn). If there is a single file containing the dumping functions then it should just be a matter of knowing you need to look into that file. (OTOH, this way the files at least have the class name in them, so it's easier to find them by just browsing the source tree)

In https://reviews.llvm.org/D48351#1168453, @clayborg wrote:

> I am fine if this helps with not pulling in the world. Seems like it should move out of Core then no? Seems weird to make the change yet keep it in the same directory.


The plan is to move RegisterValue (along with Scalar) into Utility as a follow-up. Right now I can't think of a better place than Core for the dumping functions as (at least in their current implementations) depend on pretty much everything.


https://reviews.llvm.org/D48351