Remove cast(STC) from TypeTag.semantic (#20931)

This commit is contained in:
Dennis 2025-03-01 00:44:56 +01:00 committed by GitHub
parent 2c304a4377
commit 978b108b1b
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -2985,8 +2985,14 @@ Type typeSemantic(Type type, Loc loc, Scope* sc)
Type visitTag(TypeTag mtype) Type visitTag(TypeTag mtype)
{ {
//printf("TypeTag.semantic() %s\n", mtype.toChars()); //printf("TypeTag.semantic() %s\n", mtype.toChars());
Type returnType(Type t) Type returnType(TypeTag tt)
{ {
Type t = tt.resolved;
// To make const checking work, the const STC needs to be added:
// t = t.resolved.addSTC(mtype.mod.ModToStc);
// However, this currently fails compilable/test22875.i
// Apparently there's some aliasing going on, where mutable
// versions of the type also get const applied to them.
return t.deco ? t : t.merge(); return t.deco ? t : t.merge();
} }
@ -2994,7 +3000,7 @@ Type typeSemantic(Type type, Loc loc, Scope* sc)
{ {
/* struct S s, *p; /* struct S s, *p;
*/ */
return returnType(mtype.resolved.addSTC(cast(STC) mtype.mod)); return returnType(mtype);
} }
/* Find the current scope by skipping tag scopes. /* Find the current scope by skipping tag scopes.
@ -3067,7 +3073,7 @@ Type typeSemantic(Type type, Loc loc, Scope* sc)
{ {
mtype.id = Identifier.generateId("__tag"[]); mtype.id = Identifier.generateId("__tag"[]);
declareTag(); declareTag();
return returnType(mtype.resolved.addSTC(cast(STC) mtype.mod)); return returnType(mtype);
} }
/* look for pre-existing declaration /* look for pre-existing declaration
@ -3080,7 +3086,7 @@ Type typeSemantic(Type type, Loc loc, Scope* sc)
if (mtype.tok == TOK.enum_ && !mtype.members) if (mtype.tok == TOK.enum_ && !mtype.members)
.error(mtype.loc, "`enum %s` is incomplete without members", mtype.id.toChars()); // C11 6.7.2.3-3 .error(mtype.loc, "`enum %s` is incomplete without members", mtype.id.toChars()); // C11 6.7.2.3-3
declareTag(); declareTag();
return returnType(mtype.resolved.addSTC(cast(STC) mtype.mod)); return returnType(mtype);
} }
/* A redeclaration only happens if both declarations are in /* A redeclaration only happens if both declarations are in
@ -3180,7 +3186,7 @@ Type typeSemantic(Type type, Loc loc, Scope* sc)
declareTag(); declareTag();
} }
} }
return returnType(mtype.resolved.addSTC(cast(STC) mtype.mod)); return returnType(mtype);
} }
switch (type.ty) switch (type.ty)