ilya-biryukov added a comment. I tried to find a good place to emit unresolved typos earlier (to make sure CodeGen does not ever get `TypoExpr`), but couldn't find one. Please let me know if there's some obvious place I'm missing.
Also open to suggestions for putting assertions on whether codegen is being called on `TypoExpr`s. From looking at the code, it seems that currently if the codegen eventually arrives at a `TypoExpr`, it will emit an error saying this expression is not yet supported. There might be other failure modes (computing values of exprs containing `TypoExpr`, etc), but it feels like the current state should catch most of the potential bugs there. This change seems to be an improvement over what we had before (clangd and libclang do not crash, no errors are lost) and it's very simple. So unless people object I would propose to land it even if it does not solve all of the problems around delayed exprs. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64799/new/ https://reviews.llvm.org/D64799 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
