aaron.ballman added a comment.

In D112409#3093815 <https://reviews.llvm.org/D112409#3093815>, @balazske wrote:

> Not sure if it is good to have such a test, the first and last function is 
> enough?

Yeah, that's testing an awful lot. We don't usually aim for exhaustive tests 
with these configurable lists, so long as one or two entries are covered, we 
usually assume the rest will also be fine. Might be worth cutting the test down 
a bit.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112409/new/

https://reviews.llvm.org/D112409

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

Reply via email to