[PATCH] D25496: [lit] Remove TODO

Dan Liew via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 19 03:42:21 PDT 2016


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





More information about the llvm-commits mailing list