[PATCH] D27603: Propagate -fdiagnostics-color to LLD.

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 08:47:21 PST 2016


rnk added inline comments.


================
Comment at: lib/Driver/Tools.cpp:283
+  // Add an extra parameter for LLD. If the linker is LLD, we want to
+  // propagate -fdiagnostics-color.
+  if (StringRef(TC.GetLinkerPath()).endswith("ld.lld") &&
----------------
nit: Clang's original spelling was -fcolor-diagnostics, which reads better in English. GCC made -fdiagnostics-color, which we added as an alias. It makes sense ontologically, but it doesn't read as well in a comment.


https://reviews.llvm.org/D27603





More information about the llvm-commits mailing list