[PATCH] D25496: [lit] Remove TODO

[Dan Liew via llvm-commits]

delcypher added a comment.

@modocache Sorry for the delay in taking a look at this.

This change mostly LGTM except it looks like you've dropped these TODOs

  * Move temp directory name into local test config.
  
  * Support valgrind in all configs, and LLVM style valgrind.
  
  * Support ulimit.
  
  * Create an explicit test suite object (instead of using the top-level TestingConfig object).

Is that intentional?


https://reviews.llvm.org/D25496