Changeset 6018ddb for src

Timestamp:

Feb 25, 2026, 12:52:06 AM (3 days ago)

Author:

Matthew Au-Yeung <mw2auyeu@…>

Branches:

stuck-waitfor-destruct

Children:

a0548c2

Parents:

2b3ebe5

git-author:

Matthew Au-Yeung <mw2auyeu@…> (02/25/26 00:31:10)

git-committer:

Matthew Au-Yeung <mw2auyeu@…> (02/25/26 00:52:06)

Message:

Revert "cleanup and add tests"

This reverts commit a56d544a48fa37aa9352321253b32a7ddc0159c7.

Location:

src

Files:

• AST/Expr.cpp (modified) (1 diff)
• AST/Expr.hpp (modified) (1 diff)
• Concurrency/MutexFuncHash.hpp (modified) (3 diffs)

src/AST/Expr.cpp

@@ -264 +264 @@
 }
 
-ConstantExpr * ConstantExpr::from_ulonglong( const CodeLocation & loc, unsigned long long i ) {
-        return new ConstantExpr{
-                loc, new BasicType{ BasicKind::LongLongUnsignedInt }, std::to_string( i ) + "ULL",
-                (unsigned long long)i };
-}
-
 ConstantExpr * ConstantExpr::from_string( const CodeLocation & loc, const std::string & str ) {
         const Type * charType = new BasicType( BasicKind::Char, ast::CV::Const );

src/AST/Expr.hpp

@@ -464 +464 @@
         /// Generates an integer constant of the given unsigned long int.
         static ConstantExpr * from_ulong( const CodeLocation & loc, unsigned long i );
-        /// Generates an integer constant of the given unsigned long long int.
-        static ConstantExpr * from_ulonglong( const CodeLocation & loc, unsigned long long i );
         /// Generates a string constant from the given string (char type, unquoted string).
         static ConstantExpr * from_string( const CodeLocation & loc, const std::string & string );

src/Concurrency/MutexFuncHash.hpp

@@ -18 +18 @@
 #include "AST/Decl.hpp"
 #include "AST/Expr.hpp"
+#include "AST/Type.hpp"
 #include "SymTab/Mangler.hpp"
 
@@ -26 +27 @@
 // since function pointers may differ for static inline functions.
 static inline uint64_t hashMangle( const ast::DeclWithType * decl ) {
-        std::string name = Mangle::mangleType( decl );
+        std::string name = Mangle::mangle( decl );
         uint64_t hash = 14695981039346656037ULL; // FNV offset basis
         for ( char c : name ) {
@@ -35 +36 @@
 }
 
-// Create a ConstantExpr with the hash of the mangled name.
-static inline ast::ConstantExpr * hashMangleExpr( 
-        const CodeLocation & location, const ast::DeclWithType * decl ) {
-        return ast::ConstantExpr::from_ulonglong( location, hashMangle( decl ) );
+// Create a ConstantExpr for the hash with proper ULL suffix to avoid
+// C compiler warnings about large unsigned constants.
+static inline ast::ConstantExpr * hashMangleExpr(
+                const CodeLocation & location, const ast::DeclWithType * decl ) {
+        uint64_t hash = hashMangle( decl );
+        return new ast::ConstantExpr{
+                location,
+                new ast::BasicType{ ast::BasicKind::LongLongUnsignedInt },
+                std::to_string( hash ) + "ull",
+                (unsigned long long)hash };
 }