[PATCH] D119572: Add virtual mem size when setting RSS limit on Apple platforms.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 12 04:32:36 PST 2022


fhahn abandoned this revision.
fhahn added a comment.

I did some more testing and for some benchmarks the provided default value by llvm-lit is too small. Bumping the default is not straigth-forward, as on other platforms `rlim_t` may only be a `unsigned long`. And timeit uses `atoi` to parse the argument value. I don't think this is worth all the trouble and for now I just committed a change ignoring `RLIMIT_RSS` on Apple platforms (916fce4cbbfd <https://reviews.llvm.org/rT916fce4cbbfdad9295e344b14d7017b3745001c8>) so the test-suite can be run again on Apple platforms that more strictly enforce the virtual memory limit.


Repository:
  rT test-suite

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

https://reviews.llvm.org/D119572



More information about the llvm-commits mailing list