[PATCH] D52951: [lld] Add more complete support for the INCLUDE command.

Ian Tessier via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 11 14:49:22 PDT 2018


itessier updated this revision to Diff 169311.
itessier added a comment.

I converted all the tests into .test files. I removed the empty file test in memory-include.test since it requires modifying the linker script. I could add a separate linker script in the input data directory for this case, or maybe just simplify the remaining tests by removing the empty file checks?


https://reviews.llvm.org/D52951

Files:
  ELF/ScriptParser.cpp
  test/ELF/linkerscript/memory-include.test
  test/ELF/linkerscript/output-section-include.test
  test/ELF/linkerscript/section-include.test

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52951.169311.patch
Type: text/x-patch
Size: 4892 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181011/162ee4b6/attachment.bin>


More information about the llvm-commits mailing list