[Lldb-commits] [PATCH] D54072: Add a command to dump a module's clang ast.
Zachary Turner via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Sun Nov 4 16:40:56 PST 2018
zturner added a comment.
In https://reviews.llvm.org/D54072#1286811, @zturner wrote:
> So I tried to go down this route, but alas, I still can't come up with a good way to make color output work, because the Stream is not a `StreamFile` object, it is a `StreamString` object, even when the output is going to a terminal. We could have an argument such as `--color` but perhaps it should be the default and the option should be `--no-color`.
I tried some more, and yea this is really not easy. The only `raw_ostream` in LLVM that supports color output is the `raw_fd_ostream`, so we can't even easily set up the forwarder to support any of this. Even if we configure the diagnostics engine to use color output and pass it the forwarder `raw_ostream` it still won't work, because it will just call `raw_ostream::changeColor` which does nothing, since that method is only implemented on `raw_fd_ostream`.
I'm open to suggestions, but it would be a real shame to not be able to take advantage of the built-in colorization here. Perhaps we can make this a hidden or unsupported command.
https://reviews.llvm.org/D54072
More information about the lldb-commits
mailing list