[PATCH] [dwarfdump] Pretty print location expressions and location lists
Frederic Riss
friss at apple.com
Mon Jan 5 10:22:03 PST 2015
I could commit the DWARFDebugLoc refactoring (without the pretty printing) separately, but this is about all that can be split out. This patch really has 4 parts that are reviewable separately (but that don't really make sense as separate):
- The refactoring in DWARFDebugLoc.{cpp/h}: This I could commit separately leaving the new pretty-printing out.
- The new DWARFExpression.cpp/h}: The bulk of the patch logic. Pretty self contained it interacts with the outside world through just 1 function. Can be reviewed separately.
- The integration into the DIE/DebugLoc dumpers.
- The tests updates.
Leaving out the DWO case would spare maybe 10 lines, I don't think this will really make it more digestible for you...
http://reviews.llvm.org/D6771
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list