[PATCH] D81056: [Support] Replace 'DisableColors' boolean with 'ColorMode' enum

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 5 10:01:55 PDT 2020


JDevlieghere added a comment.

With the new approach this change isn't strictly necessary for D81058 <https://reviews.llvm.org/D81058> to work, but I still think it's an (small) improvement over the status quo. I'm happy to either land or abandon the patch, whatever the community feels is most desirable.


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

https://reviews.llvm.org/D81056





More information about the llvm-commits mailing list