mikerice1969 wrote: I mentioned the assert just to make the point that setSubExpr is written so it doesn't return a nullptr. So it is not expected and the deference is ok.
I went ahead with this change but feel free to update if you like. But adding a return when nullptr is seen here would confuse the code and static verifiers will complain that the return can never happen. https://github.com/llvm/llvm-project/pull/85473 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
