[PATCH] D83834: Add test utility 'extract'
Chris Lattner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 24 08:59:13 PDT 2020
lattner added a comment.
I don't see any reason to prefer a short name here, this isn't a simple too like 'opt' it takes command line flags etc.
Unrelatedly, I don't agree with your rationale for having a separate tool. You're right that moving files to an Inputs directory moves them further away, but that isn't what I was suggesting. Also, there is great precedent across the testsuite for this, and the proposed tool isn't a general solution to the problem (e.g. binary files etc).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83834/new/
https://reviews.llvm.org/D83834
More information about the llvm-commits
mailing list