[PATCH] D30107: Make DynamicLibrary::getPermanentLibrary have a defined ordering.

Vassil Vassilev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 6 08:10:41 PST 2017


v.g.vassilev added a comment.

It looks like this patch is doing a lot more than what is described in the summary. I think it would make the review easier if you split the patch. I'd imagine that fixing the order of iteration is fairly minimal change and I'd recommend to start from there by opening another review request with just that.


https://reviews.llvm.org/D30107





More information about the llvm-commits mailing list