[PATCH] D65191: [llvm-objdump] Implement highlighting

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: llvm/lib/Support/WithColor.cpp:107
+
+bool WithColor::colorsAvailable(raw_ostream &OS) {
   if (UseColor == cl::BOU_UNSET)
----------------
There is no need to split.


================
Comment at: llvm/tools/llvm-objdump/llvm-objdump.cpp:2290
+  ColorsEnabled =
+      WithColor::isColorOptionSet() && WithColor::colorsAvailable(outs());
+
----------------
`WithColor::isColorOptionSet() && outs().has_colors()`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65191/new/

https://reviews.llvm.org/D65191