[PATCH] D84885: Make test use a smaller file

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 29 13:04:01 PDT 2020


MaskRay added a comment.

That patch fell out of my radar when it was committed. I would slightly change the code construct the test file used. Fixed in ed7bde0e4b40cbf8a7c833fd8240c957fcda176e <https://reviews.llvm.org/rGed7bde0e4b40cbf8a7c833fd8240c957fcda176e>

We should use `-o /dev/null` if the output is not needed and use a section of different permissions. .text_low & .text_overflow are by default in the same segment and can make the segment have very large p_filesz.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84885



More information about the llvm-commits mailing list