[PATCH] D47540: [lld] Enable Visual Studio compatible output

Chris Jackson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 1 09:09:08 PDT 2018


chrisjackson updated this revision to Diff 149497.
chrisjackson added a comment.

Thanks for reviewing Rui, I've made the changes as described in my responses to your comments. Primarily:

- removed the extra set of LogName
- made clearer the extraction of the linker executable name from Args
- Made a correction to print() in ErrorHandler
- Removed unnecessary else in inputfiles.cpp


https://reviews.llvm.org/D47540

Files:
  Common/ErrorHandler.cpp
  ELF/Driver.cpp
  ELF/Driver.h
  ELF/DriverUtils.cpp
  ELF/InputFiles.cpp
  ELF/Options.td
  ELF/Relocations.cpp
  include/lld/Common/ErrorHandler.h
  test/ELF/vs-diagnostics.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47540.149497.patch
Type: text/x-patch
Size: 10344 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180601/d2b6e856/attachment.bin>


More information about the llvm-commits mailing list