[PATCH] D25496: [lit] Remove TODO

Brian Gesiak via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 19 13:32:23 PDT 2016


modocache added a comment.

- "Move temp directory name into local test config" has been moved to https://llvm.org/bugs/show_bug.cgi?id=30673. Based on the comment on that bug, it may already be done!
- "Support ulimit" has been moved to https://llvm.org/bugs/show_bug.cgi?id=30672.

As for the others:

- I chose not to migrate "Create an explicit test suite object (instead of using the top-level TestingConfig object)" because I wasn't sure it was still relevant. It also seems like an implementation detail to me.
- I'd appreciate it if you could create a problem report for "Support valgrind in all configs, and LLVM style valgrind", @delcypher! I'm not sure I understand what is and isn't supported.


https://reviews.llvm.org/D25496





More information about the llvm-commits mailing list