[PATCH] D53031: [LLD] [MinGW] Expand comment for MinGW driver.

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo updated this revision to Diff 180324.
mstorsjo retitled this revision from "Expand comment for MinGW driver." to "[LLD] [MinGW] Expand comment for MinGW driver.".
mstorsjo added a reviewer: smeenai.
mstorsjo added a comment.

Corrected @smeenai's review comments.


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

https://reviews.llvm.org/D53031

Files:
  MinGW/Driver.cpp


Index: MinGW/Driver.cpp
===================================================================
--- MinGW/Driver.cpp
+++ MinGW/Driver.cpp
@@ -6,9 +6,27 @@
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
-///
-/// GNU ld style linker driver for COFF currently supporting mingw-w64.
-///
+//
+// MinGW is a GNU development environment for Windows. It consists of GNU
+// tools such as GCC and GNU ld. Unlike Cygwin, there's no POSIX-compatible
+// layer, as it aims to be a native development toolchain.
+//
+// lld/MinGW is a drop-in replacement for GNU ld/MinGW.
+//
+// Being a native development tool, a MinGW linker is not very different from
+// Microsoft link.exe, so a MinGW linker can be implemented as a thin wrapper
+// for lld/COFF. This driver takes Unix-ish command line options, translates
+// them to Windows-ish ones, and then passes them to lld/COFF.
+//
+// When this driver calls the lld/COFF driver, it passes a hidden option
+// "-lldmingw" along with other user-supplied options, to run the lld/COFF
+// linker in "MinGW mode".
+//
+// There are subtle differences between MS link.exe and GNU ld/MinGW, and GNU
+// ld/MinGW implements a few GNU-specific features. Such features are directly
+// implemented in lld/COFF and enabled only when the linker is running in MinGW
+// mode.
+//
 //===----------------------------------------------------------------------===//
 
 #include "lld/Common/Driver.h"


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53031.180324.patch
Type: text/x-patch
Size: 1499 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190104/184547f2/attachment.bin>