[PATCH] D31614: Simplify test a bit
Reid Kleckner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 5 11:44:19 PDT 2017
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
Sure, this is simpler, and I'm not concerned about FileCheck blowing out a 4MB stack.
Do you want to add `REQUIRES: shell` in this change as well, since this test probably still fails with "ulimit: command not found" when setting `LIT_USE_INTERNAL_SHELL=1`?
https://reviews.llvm.org/D31614
More information about the llvm-commits
mailing list