efriedma-quic wrote: > That said, this patch might be worth having in that branch AS WELL to cover > any other missed cases. I would be OK having this patch HERE only on that > branch or replaced with an assert in main.
#128409 should cover all the cases this patch would cover. I'm a little concerned #128409 still isn't broad enough... but not in a way that would trigger this check. I'll continue discussion there. https://github.com/llvm/llvm-project/pull/127525 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
