[PATCH] D66717: [ELF] Do not ICF two sections with different output sections (by SECTIONS commands)

[Fangrui Song via Phabricator via llvm-commits]

MaskRay updated this revision to Diff 217113.
MaskRay marked 4 inline comments as done.
MaskRay added a comment.

Delete a redundant dyn_cast<OutputSection>
Add more comments to ICF<ELFT>::run()


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D66717

Files:
  ELF/Driver.cpp
  ELF/ICF.cpp
  ELF/LinkerScript.cpp
  ELF/LinkerScript.h
  ELF/Writer.cpp
  ELF/Writer.h
  test/ELF/linkerscript/early-assign-symbol.s
  test/ELF/linkerscript/icf-output-sections.s
  test/ELF/linkerscript/subalign.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D66717.217113.patch
Type: text/x-patch
Size: 16374 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190826/fa10ebb2/attachment.bin>