bd1976bris wrote: > LGTM. I noted a couple of redundant checks, but if you think it's better for > them to be explicit, I'm okay with keeping them.
Thanks. I think in this case it makes it more obvious what is primarily been checked in these cases to have the check explicit. https://github.com/llvm/llvm-project/pull/92091 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
