<br><br><div class="gmail_quote">On Mon Jan 05 2015 at 4:06:31 PM Adrian Prantl <<a href="mailto:aprantl@apple.com">aprantl@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In <a href="http://reviews.llvm.org/D6852#105642" target="_blank">http://reviews.llvm.org/D6852#<u></u>105642</a>, @echristo wrote:<br>
<br>
> Definitely want it.<br>
><br>
> Two things:<br>
><br>
> a) Might want to make it an option?<br>
<br>
<br>
With this implementation colorizing is depending on the terminal type: Invoking llvm-dwarfdump results in color output, whereas llvm-dwarfdump | less will be monochrome.<br>
The frontend provides an -fdiagnostics-color option that is on by default. We could do something similar here; If we do this; it should override the terminal detection, such that llvm-dwarfdump --color | less produces color output no matter what.<br>
<br></blockquote><div><br></div><div>:) This is what I'd like personally.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> b) Formatting for the SyntaxHighlighting.h file.<br>
<br>
><br>
<br>
> Is there any way we can piggy back more off the existing color support? (Not that this is complicated and I've not looked at it at all)<br>
<br>
<br>
Is there more/better color support somewhere that we could re-use? I only looked at the AST dumper in clang so far, which just hardcodes the colors.<br>
<br></blockquote><div><br></div><div>I just figured having one unified "put some color on it" type of interface. It's not necessary, I was just curious.</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
REPOSITORY<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D6852" target="_blank">http://reviews.llvm.org/D6852</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
</blockquote></div>