Changeset 53f4b55

Timestamp:

Feb 12, 2025, 4:12:04 PM (7 months ago)

Author:

Andrew Beach <ajbeach@…>

Branches:

master

Children:

c341b57

Parents:

5e8d75bb

Message:

Remove the untyped constructor for (typed) OffsetofExpr. The one time it was used may not need to create a typed expression but we have all the information to do so.

Location:

src

Files:

• AST/Expr.cpp (modified) (2 diffs)
• AST/Expr.hpp (modified) (1 diff)
• GenPoly/Box.cpp (modified) (1 diff)

src/AST/Expr.cpp

@@ -301 +301 @@
 // --- OffsetofExpr
 
-OffsetofExpr::OffsetofExpr( const CodeLocation & loc, const Type * ty, const DeclWithType * mem )
-: OffsetofExpr( loc, ty, mem, nullptr ) {}
-
 OffsetofExpr::OffsetofExpr( const CodeLocation & loc, const Type * ty, const DeclWithType * mem, const Type * res )
 : Expr( loc, res ), type( ty ), member( mem ) {
@@ -324 +321 @@
 
 // --- CommaExpr
+
 bool CommaExpr::get_lvalue() const {
         // This is wrong by C, but the current implementation uses it.

src/AST/Expr.hpp

@@ -539 +539 @@
         readonly<DeclWithType> member;
 
-        OffsetofExpr( const CodeLocation & loc, const Type * ty, const DeclWithType * mem );
         OffsetofExpr( const CodeLocation & loc, const Type * ty, const DeclWithType * mem, const Type * res );
 

src/GenPoly/Box.cpp

@@ -2002 +2002 @@
                         new ast::OffsetofExpr( expr->location,
                                 deepCopy( type ),
-                                memberDecl
+                                memberDecl,
+                                getLayoutType( transUnit() )
                         )
                 ) );