https://github.com/mizvekov approved this pull request.

LGTM, I agree that we currently don't use it, so we might as well remove it.

But the overall situation is all wrong, we shouldn't be storing fully formed 
diagnostic strings in the AST, we should be storing them as PartialDiagnostics.

But making that right requires bigger changes.

https://github.com/llvm/llvm-project/pull/98654
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to