[PATCH] D23156: [ADT] Stop trying to test every combination of values in a triple in every permutation.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 5 07:59:13 PDT 2016


chandlerc added a comment.

In https://reviews.llvm.org/D23156#506829, @rengolin wrote:

> I'm still looking at the code and the comments, but wouldn't it be enough for 99% of the cases to test it against "unknown" on all other fields? I mean, there are a few problematic combinations, but they require specialised knowledge, and neither the old test (nor the new one) are checking them.


I think that's exactly what the patch does?


https://reviews.llvm.org/D23156





More information about the llvm-commits mailing list