[clang] [clang] Fix-it hint for `++this` -> `++*this` when deref is modifiable (PR #94159)

via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 7 08:40:33 PDT 2024


================
@@ -13367,6 +13367,21 @@ static void DiagnoseConstAssignment(Sema &S, const Expr *E,
         if (!DiagnosticEmitted) {
           S.Diag(Loc, diag::err_typecheck_assign_const)
               << ExprRange << ConstVariable << VD << VD->getType();
+          ExprResult Deref;
+          Expr *TE = const_cast<Expr *>(E);
----------------
Sirraide wrote:

If it’s easier to use `const_cast` here than to change the parameter to be non-`const`, then that’s fine imo. Our const-correctness w/ AST nodes is all over the place anyway, so while not ‘ideal’, it’s not the end of the world imo

https://github.com/llvm/llvm-project/pull/94159


More information about the cfe-commits mailing list