[PATCH] D84885: Make test use a smaller file

[Douglas Yung via Phabricator via llvm-commits]

dyung created this revision.
dyung added reviewers: stefanp, MaskRay, sfertile.
dyung added projects: lld, LLVM.
Herald added subscribers: llvm-commits, kbarton, nemanjai, emaste.
Herald added a reviewer: espindola.
dyung requested review of this revision.
Herald added a subscriber: wuzish.

In our internal build bot, we were seeing random test failures due to running out of disk space which we eventually traced back to this test which tries to create a large file to test with. This was being written to disk and eating most if not all of the 59GB of free space we had remaining on the machine. And while this test itself did not fail, it caused other tests running at the same time to fail due to being unable to write their output to the disk.

It was discovered that the test should still work with  a smaller value which also avoids creating as large of a file. I've run the LLD tests with this change locally multiple times and it seems to avoid the problem we were encountering.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D84885

Files:
  lld/test/ELF/ppc64-reloc-pcrel34-overflow.s


Index: lld/test/ELF/ppc64-reloc-pcrel34-overflow.s
===================================================================
--- lld/test/ELF/ppc64-reloc-pcrel34-overflow.s
+++ lld/test/ELF/ppc64-reloc-pcrel34-overflow.s
@@ -1,7 +1,7 @@
 # REQUIRES: ppc
 # RUN: echo 'SECTIONS { \
 # RUN:       .text_low 0x10010000: { *(.text_low) } \
-# RUN:       .text_overflow 0x1000000000 : { *(.text_overflow) } \
+# RUN:       .text_overflow 0x210010000 : { *(.text_overflow) } \
 # RUN:       }' > %t.script
 
 # RUN: llvm-mc -filetype=obj -triple=powerpc64le %s -o %t.o


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D84885.281702.patch
Type: text/x-patch
Size: 557 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200729/66890bd4/attachment.bin>