[Lldb-commits] [PATCH] RFC: Proposed change in the disassembly default format in lldb

Zachary Turner zturner at google.com
Wed Feb 11 23:26:14 PST 2015


Also, would the 'disassembly-format' setting be to switch betwen old style and new style?  Or would it just be toggle between displaying the function name or not displaying the function name?  A toggle between displaying and not displaying the function name seems the best to me.  This makes the behavior more straightforward and reduces the amount of code to maintain, and I think better addresses the issue at hand: Whether a line is going to be readable due to a too-long function name or not.   If your original plan was for the 'disassembly-format' setting to control old style vs. new style, then I would vote for an additional option called 'disassembly-function-names' or something similar.  There's plenty of times I'm debugging C code with short function names and I don't want to pay the penalty for something that isn't an issue.

In the future, I have ideas for a fully customizable disassembly format that would allow you to specify a format string with placeholders, then people could arrange the disassembly output however they wanted.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D7578

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the lldb-commits mailing list