zahiraam wrote: > This is not needed IMO. This is a simple matcher, `op` is expected to always > be bound. > > One might say that it can't hurt to add extra checks, but I don't think > that's good justification for adding more code / bloat, even if in this case > it's a single line. Still, 5% more code for this check counting non-empty > non-comment lines! > > Happy to disagree if any of the maintainers think otherwise.
The sanitizer is complaining about the possibility of having `Operator` be null. `op` might always be bound. But it might not be a `UnaryOperator`, no? https://github.com/llvm/llvm-project/pull/138283 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits