dcoughlin added a comment.

In https://reviews.llvm.org/D27409#614601, @NoQ wrote:

> Not sure we need to stay merged with `retain-release-arc.m`, as we're not 
> really reusing many declarations across these files.


I find it more helpful to organize tests around functionality ('retain count 
checker') than around bugs ('dispatch data leak', 'PRxyzab.c').  Organizing 
around functionality helps future contributors and maintainers know where to 
add a new test and where to look for existing tests. In my view this critical 
for developers who are new to the project or who are learning a new subsystem. 
It also typically reduces the number of 'clang' RUN lines, which have a cost in 
maintenance and running time.


https://reviews.llvm.org/D27409



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to