[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