aaron.ballman added a comment.

In D142401#4307493 <https://reviews.llvm.org/D142401#4307493>, @cor3ntin wrote:

> I don't think this is actually testable, ultimately we may run out of stack
> any way, the warning (when emitted) even say so.

Okay, that's a fair point and makes my suggestion invalid. I think you're 
right, testing this consistently will be tricky -- LGTM once the test is 
removed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142401

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

Reply via email to