[PATCH] D44438: [ELF] - Convert overlapping-sections.s testcase to x86 and cleanup.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 13 09:54:24 PDT 2018


grimar created this revision.
grimar added reviewers: arichardson, ruiu, espindola.
Herald added subscribers: sdardis, emaste.

Patch do the following change:

- Test case was converted from MIPS to x86. Not sure why it was done as MIPS initially, but since it had TODO comment saying we can switch to x86, I did that.

- Does not seem to make sense to check "good" output in this test case. We check that our output is good in many other tests and intention of this one is only to check we are still able to report overlaps and/or produce broken output with overlaps I think. So I removed that part of the test.


https://reviews.llvm.org/D44438

Files:
  test/ELF/linkerscript/overlapping-sections.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44438.138214.patch
Type: text/x-patch
Size: 9355 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180313/8759790d/attachment.bin>


More information about the llvm-commits mailing list