[PATCH] D33880: COFF: Introduce LD shim around LINK
Rui Ueyama via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 11 18:34:04 PDT 2017
ruiu added inline comments.
================
Comment at: lld/include/lld/Driver/Driver.h:21
+bool ldshim(llvm::ArrayRef<const char *> Args,
+ llvm::raw_ostream &Diag = llvm::errs());
}
----------------
Indentation. Please use clang-format-diff before sending patches so that human reviewers can focus on meaning of the code instead of formatting.
================
Comment at: lld/lib/Shim/COFFLdShim.cpp:1
+//===- lib/Driver/COFFLdShim.cpp ------------------------------------------===//
+//
----------------
ruiu wrote:
> martell wrote:
> > Location Okay?
> See the comment above.
>
> Rename this file `MingwDriver.cpp`.
Please move this file as I wrote above.
Repository:
rL LLVM
https://reviews.llvm.org/D33880
More information about the llvm-commits
mailing list