[lld] r336179 - [ELF] - Revert huge-temporary-file.s changed in r336129

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 3 01:30:56 PDT 2018


Author: grimar
Date: Tue Jul  3 01:30:55 2018
New Revision: 336179

URL: http://llvm.org/viewvc/llvm-project?rev=336179&view=rev
Log:
[ELF] - Revert huge-temporary-file.s changed in r336129

Comment in the test case says that:

## This inputs previously created a 4gb temporarily file under 32 bit
## configuration. Issue was fixed. There is no clean way to check that from here.
## This testcase added for documentation purposes.

The intention of the test was to create such huge file
in case if our code will be broken again.
And currently it documents we do not create huge outputs.

r336129 changed -o to /dev/null and broke the intentions of the test case.

Modified:
    lld/trunk/test/ELF/linkerscript/huge-temporary-file.s

Modified: lld/trunk/test/ELF/linkerscript/huge-temporary-file.s
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/test/ELF/linkerscript/huge-temporary-file.s?rev=336179&r1=336178&r2=336179&view=diff
==============================================================================
--- lld/trunk/test/ELF/linkerscript/huge-temporary-file.s (original)
+++ lld/trunk/test/ELF/linkerscript/huge-temporary-file.s Tue Jul  3 01:30:55 2018
@@ -1,7 +1,7 @@
 # REQUIRES: x86
 # RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %s -o %t
 # RUN: echo "SECTIONS { .text 0x2000 : {. = 0x10 ; *(.text) } }" > %t.script
-# RUN: not ld.lld %t --script %t.script -o /dev/null
+# RUN: not ld.lld %t --script %t.script -o %t1
 
 ## This inputs previously created a 4gb temporarily file under 32 bit
 ## configuration. Issue was fixed. There is no clean way to check that from here.




More information about the llvm-commits mailing list