vbvictor wrote: > Based on the experiment in [#187454 > (comment)](https://github.com/llvm/llvm-project/pull/187454#discussion_r2964303225), > I will stay with the `if(..) return;` way for now before we figure out > what's wrong with `unchecked-optional-access`.
I think we should merge with failing tidy in this case or turn off this check temporary. I'm pretty sure we have 5+ more places where we use `assert()` as "defence mechanic" and those must be broken as well. https://github.com/llvm/llvm-project/pull/187454 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
