[PATCH] D39259: [lld] unified COFF and ELF error handling on new Common/ErrorHandler

Bob Haarman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 25 15:02:53 PDT 2017


inglorion updated this revision to Diff 120324.
inglorion added a comment.

- Removed ErrorHandler parameter to handleColorDiagnostics.
- Cut number of constructors for ErrorHandler down to one.


https://reviews.llvm.org/D39259

Files:
  lld/COFF/CMakeLists.txt
  lld/COFF/Chunks.cpp
  lld/COFF/Config.h
  lld/COFF/Driver.cpp
  lld/COFF/DriverUtils.cpp
  lld/COFF/Error.cpp
  lld/COFF/Error.h
  lld/COFF/ICF.cpp
  lld/COFF/InputFiles.cpp
  lld/COFF/LTO.cpp
  lld/COFF/MapFile.cpp
  lld/COFF/MinGW.cpp
  lld/COFF/PDB.cpp
  lld/COFF/SymbolTable.cpp
  lld/COFF/Symbols.cpp
  lld/COFF/Writer.cpp
  lld/Common/CMakeLists.txt
  lld/Common/ErrorHandler.cpp
  lld/ELF/Arch/AArch64.cpp
  lld/ELF/Arch/AMDGPU.cpp
  lld/ELF/Arch/ARM.cpp
  lld/ELF/Arch/AVR.cpp
  lld/ELF/Arch/Mips.cpp
  lld/ELF/Arch/MipsArchTree.cpp
  lld/ELF/Arch/PPC.cpp
  lld/ELF/Arch/PPC64.cpp
  lld/ELF/Arch/SPARCV9.cpp
  lld/ELF/Arch/X86.cpp
  lld/ELF/Arch/X86_64.cpp
  lld/ELF/CMakeLists.txt
  lld/ELF/Config.h
  lld/ELF/Driver.cpp
  lld/ELF/DriverUtils.cpp
  lld/ELF/EhFrame.cpp
  lld/ELF/Error.cpp
  lld/ELF/Error.h
  lld/ELF/InputFiles.cpp
  lld/ELF/InputFiles.h
  lld/ELF/InputSection.cpp
  lld/ELF/LTO.cpp
  lld/ELF/Relocations.cpp
  lld/ELF/ScriptLexer.cpp
  lld/ELF/ScriptParser.cpp
  lld/ELF/Strings.cpp
  lld/ELF/SymbolTable.cpp
  lld/ELF/Symbols.cpp
  lld/ELF/SyntheticSections.cpp
  lld/ELF/Target.cpp
  lld/ELF/Target.h
  lld/ELF/Thunks.cpp
  lld/ELF/Writer.cpp
  lld/include/lld/Common/ErrorHandler.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39259.120324.patch
Type: text/x-patch
Size: 56059 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171025/69a498c2/attachment.bin>


More information about the llvm-commits mailing list