CodeSpecifiers fixes

base/components/code_serialization.cpp

@@ -38,13 +38,13 @@ void body_to_strbuilder_export( CodeBody body, StrBuilder* result )
 	GEN_ASSERT(result != nullptr);
 	strbuilder_append_fmt( result, "export\n{\n" );
 
-	Code curr = body->Front;
+	Code curr = cast(Code, body);
 	s32  left = body->NumEntries;
 	while ( left-- )
 	{
 		code_to_strbuilder_ref(curr, result);
 		// strbuilder_append_fmt( result, "%SB", code_to_strbuilder(curr) );
-		curr = curr->Next;
+		++curr;
 	}
 
 	strbuilder_append_fmt( result, "};\n" );

base/components/code_types.hpp

@@ -252,8 +252,8 @@ struct CodeSpecifiers
 #if ! GEN_C_LIKE_CPP
 	Using_Code( CodeSpecifiers );
 	bool       append( Specifier spec )            { return specifiers_append(* this, spec); }
-	s32        has( Specifier spec )               { return specifiers_has(* this, spec); }
-	s32        index_of( Specifier spec )          { return specifiers_index_of(* this, spec); }
+	bool       has( Specifier spec )               { return specifiers_has(* this, spec); }
+	s32        index_of(Specifier spec)            { return specifiers_index_of(* this, spec); }
 	s32        remove( Specifier to_remove )       { return specifiers_remove(* this, to_remove); }
 	StrBuilder to_strbuilder()                     { return specifiers_to_strbuilder(* this ); }
 	void       to_strbuilder( StrBuilder& result ) { return specifiers_to_strbuilder_ref(* this, & result); }
@@ -1075,7 +1075,8 @@ forceinline StrBuilder to_strbuilder(CodeParams params )                       {
 forceinline void       to_strbuilder(CodeParams params, StrBuilder& result )   { return params_to_strbuilder_ref(params, & result); }
   
 forceinline bool       append       (CodeSpecifiers specifiers, Specifier spec)       { return specifiers_append(specifiers, spec); }
-forceinline s32        has          (CodeSpecifiers specifiers, Specifier spec)       { return specifiers_has(specifiers, spec); }
+forceinline bool       has          (CodeSpecifiers specifiers, Specifier spec)       { return specifiers_has(specifiers, spec); }
+forceinline s32        index_of     (CodeSpecifiers specifiers, Specifier spec)       { return specifiers_index_of(specifiers, spec); }
 forceinline s32        remove       (CodeSpecifiers specifiers, Specifier to_remove ) { return specifiers_remove(specifiers, to_remove); }
 forceinline StrBuilder to_strbuilder(CodeSpecifiers specifiers)                       { return specifiers_to_strbuilder(specifiers); }
 forceinline void       to_strbuilder(CodeSpecifiers specifiers, StrBuilder& result)   { return specifiers_to_strbuilder_ref(specifiers, & result);  }

base/components/inlines.hpp

@@ -38,7 +38,7 @@ void body_to_strbuilder_ref( CodeBody body, StrBuilder* result )
 	{
 		code_to_strbuilder_ref(curr, result);
 		// strbuilder_append_fmt( result, "%SB", code_to_strbuilder(curr) );
-		curr = curr->Next;
+		++curr;
 	}
 }
 

base/components/interface.cpp

@@ -283,15 +283,6 @@ void init(Context* ctx)
 		ctx->InitSize_Fallback_Allocator_Bucket_Size = megabytes(8);
 	}
 
-	if (ctx->InitSize_StrCacheTable == 0)
-	{
-		ctx->InitSize_StrCacheTable = kilobytes(8);
-	}
-	if (ctx->InitSize_MacrosTable == 0)
-	{
-		ctx->InitSize_MacrosTable = kilobytes(8);
-	}
-
 	// Override the current context (user has to put it back if unwanted).
 	_ctx = ctx;
 
@@ -320,11 +311,11 @@ void init(Context* ctx)
 	}
 	// Setup the hash tables
 	{
-		ctx->StrCache = hashtable_init_reserve(StrCached, ctx->Allocator_DyanmicContainers, ctx->InitSize_StrCacheTable);
+		ctx->StrCache = hashtable_init(StrCached, ctx->Allocator_DyanmicContainers);
 		if ( ctx->StrCache.Entries == nullptr )
 			GEN_FATAL( "gen::init: Failed to initialize the StringCache");
 
-		ctx->Macros = hashtable_init_reserve(Macro, ctx->Allocator_DyanmicContainers, ctx->InitSize_MacrosTable);
+		ctx->Macros = hashtable_init(Macro, ctx->Allocator_DyanmicContainers);
 		if (ctx->Macros.Hashes == nullptr || ctx->Macros.Entries == nullptr) {
 			GEN_FATAL( "gen::init: Failed to initialize the PreprocessMacros table" );
 		}

base/components/interface.hpp

@@ -64,9 +64,6 @@ struct Context
 	u32 InitSize_LexerTokens;
 	u32 SizePer_StringArena;
 
-	u32 InitSize_StrCacheTable;
-	u32 InitSize_MacrosTable;
-
 // TODO(Ed): Symbol Table
 	// Keep track of all resolved symbols (naemspaced identifiers)
 

base/components/interface.upfront.cpp

@@ -473,10 +473,8 @@ CodeComment def_comment( Str content )
 	return (CodeComment) result;
 }
 
-CodeConstructor def_constructor( Opts_def_constructor opt )
+CodeConstructor def_constructor( Opts_def_constructor p )
 {
-	Opts_def_constructor p = get_optional(opt);
-
 	if ( p.params && p.params->Type != CT_Parameters ) {
 		log_failure("gen::def_constructor: params must be of Parameters type - %s", code_debug_str((Code)p.params));
 		GEN_DEBUG_TRAP();
@@ -512,10 +510,8 @@ CodeConstructor def_constructor( Opts_def_constructor opt )
 	return result;
 }
 
-CodeClass def_class( Str name, Opts_def_struct opt )
+CodeClass def_class( Str name, Opts_def_struct p )
 {
-	Opts_def_struct p = get_optional(opt);
-
 	if ( ! name_check( def_class, name ) ) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;
@@ -565,10 +561,8 @@ CodeClass def_class( Str name, Opts_def_struct opt )
 	return result;
 }
 
-CodeDefine def_define( Str name, MacroType type, Opts_def_define opt )
+CodeDefine def_define( Str name, MacroType type, Opts_def_define p )
 {
-	Opts_def_define p = get_optional(opt);
-
 	if ( ! name_check( def_define, name ) ) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;
@@ -591,10 +585,8 @@ CodeDefine def_define( Str name, MacroType type, Opts_def_define opt )
 	return result;
 }
 
-CodeDestructor def_destructor( Opts_def_destructor opt )
+CodeDestructor def_destructor( Opts_def_destructor p )
 {
-	Opts_def_destructor p = get_optional(opt);
-
 	if ( p.specifiers && p.specifiers->Type != CT_Specifiers ) {
 		log_failure( "gen::def_destructor: specifiers was not a 'Specifiers' type: %s", code_debug_str(p.specifiers) );
 		GEN_DEBUG_TRAP();
@@ -627,10 +619,8 @@ CodeDestructor def_destructor( Opts_def_destructor opt )
 	return result;
 }
 
-CodeEnum def_enum( Str name, Opts_def_enum opt )
+CodeEnum def_enum( Str name, Opts_def_enum p )
 {
-	Opts_def_enum p = get_optional(opt);
-
 	if ( ! name_check( def_enum, name ) ) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;
@@ -752,10 +742,8 @@ CodeFriend def_friend( Code declaration )
 	return result;
 }
 
-CodeFn def_function( Str name, Opts_def_function opt )
+CodeFn def_function( Str name, Opts_def_function p )
 {
-	Opts_def_function p = get_optional(opt);
-
 	if ( ! name_check( def_function, name )) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;
@@ -814,10 +802,8 @@ CodeFn def_function( Str name, Opts_def_function opt )
 	return result;
 }
 
-CodeInclude def_include( Str path, Opts_def_include opt )
+CodeInclude def_include( Str path, Opts_def_include p )
 {
-	Opts_def_include p = get_optional(opt);
-
 	if ( path.Len <= 0 || path.Ptr == nullptr ) {
 		log_failure( "gen::def_include: Invalid path provided - %d" );
 		GEN_DEBUG_TRAP();
@@ -835,10 +821,8 @@ CodeInclude def_include( Str path, Opts_def_include opt )
 	return result;
 }
 
-CodeModule def_module( Str name, Opts_def_module opt )
+CodeModule def_module( Str name, Opts_def_module p )
 {
-	Opts_def_module p = get_optional(opt);
-
 	if ( ! name_check( def_module, name )) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;
@@ -851,10 +835,8 @@ CodeModule def_module( Str name, Opts_def_module opt )
 	return result;
 }
 
-CodeNS def_namespace( Str name, CodeBody body, Opts_def_namespace opt )
+CodeNS def_namespace( Str name, CodeBody body, Opts_def_namespace p )
 {
-	Opts_def_namespace p = get_optional(opt);
-
 	if ( ! name_check( def_namespace, name )) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;
@@ -877,10 +859,8 @@ CodeNS def_namespace( Str name, CodeBody body, Opts_def_namespace opt )
 	return result;
 }
 
-CodeOperator def_operator( Operator op, Str nspace, Opts_def_operator opt )
+CodeOperator def_operator( Operator op, Str nspace, Opts_def_operator p )
 {
-	Opts_def_operator p = get_optional(opt);
-
 	if ( p.attributes && p.attributes->Type != CT_PlatformAttributes ) {
 		log_failure( "gen::def_operator: PlatformAttributes was provided but its not of attributes type: %s", code_debug_str(p.attributes) );
 		GEN_DEBUG_TRAP();
@@ -946,10 +926,8 @@ CodeOperator def_operator( Operator op, Str nspace, Opts_def_operator opt )
 	return result;
 }
 
-CodeOpCast def_operator_cast( CodeTypename type, Opts_def_operator_cast opt )
+CodeOpCast def_operator_cast( CodeTypename type, Opts_def_operator_cast p )
 {
-	Opts_def_operator_cast p = get_optional(opt);
-
 	if ( ! null_check( def_operator_cast, type )) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;
@@ -981,10 +959,8 @@ CodeOpCast def_operator_cast( CodeTypename type, Opts_def_operator_cast opt )
 	return result;
 }
 
-CodeParams def_param( CodeTypename type, Str name, Opts_def_param opt )
+CodeParams def_param( CodeTypename type, Str name, Opts_def_param p )
 {
-	Opts_def_param p = get_optional(opt);
-
 	if ( ! name_check( def_param, name ) || ! null_check( def_param, type ) ) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;
@@ -1058,10 +1034,8 @@ CodeSpecifiers def_specifier( Specifier spec )
 	return result;
 }
 
-CodeStruct def_struct( Str name, Opts_def_struct opt )
+CodeStruct def_struct( Str name, Opts_def_struct p )
 {
-	Opts_def_struct p = get_optional(opt);
-
 	if ( p.attributes && p.attributes->Type != CT_PlatformAttributes ) {
 		log_failure( "gen::def_struct: attributes was not a `PlatformAttributes` type - %s", code_debug_str(cast(Code, p.attributes)) );
 		GEN_DEBUG_TRAP();
@@ -1102,10 +1076,8 @@ CodeStruct def_struct( Str name, Opts_def_struct opt )
 	return result;
 }
 
-CodeTemplate def_template( CodeParams params, Code declaration, Opts_def_template opt )
+CodeTemplate def_template( CodeParams params, Code declaration, Opts_def_template p )
 {
-	Opts_def_template p = get_optional(opt);
-
 	if ( ! null_check( def_template, declaration ) ) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;
@@ -1136,10 +1108,8 @@ CodeTemplate def_template( CodeParams params, Code declaration, Opts_def_templat
 	return result;
 }
 
-CodeTypename def_type( Str name, Opts_def_type opt )
+CodeTypename def_type( Str name, Opts_def_type p )
 {
-	Opts_def_type p = get_optional(opt);
-
 	if ( ! name_check( def_type, name )) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;
@@ -1173,10 +1143,8 @@ CodeTypename def_type( Str name, Opts_def_type opt )
 	return result;
 }
 
-CodeTypedef def_typedef( Str name, Code type, Opts_def_typedef opt )
+CodeTypedef def_typedef( Str name, Code type, Opts_def_typedef p )
 {
-	Opts_def_typedef p = get_optional(opt);
-
 	if ( ! null_check( def_typedef, type ) ) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;
@@ -1238,10 +1206,8 @@ CodeTypedef def_typedef( Str name, Code type, Opts_def_typedef opt )
 	return result;
 }
 
-CodeUnion def_union( Str name, CodeBody body, Opts_def_union opt )
+CodeUnion def_union( Str name, CodeBody body, Opts_def_union p )
 {
-	Opts_def_union p = get_optional(opt);
-
 	if ( ! null_check( def_union, body ) ) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;
@@ -1267,10 +1233,8 @@ CodeUnion def_union( Str name, CodeBody body, Opts_def_union opt )
 	return result;
 }
 
-CodeUsing def_using( Str name, CodeTypename type, Opts_def_using opt )
+CodeUsing def_using( Str name, CodeTypename type, Opts_def_using p )
 {
-	Opts_def_using p = get_optional(opt);
-
 	if ( ! name_check( def_using, name ) || null_check( def_using, type ) ) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;
@@ -1310,10 +1274,8 @@ CodeUsing def_using_namespace( Str name )
 	return result;
 }
 
-CodeVar def_variable( CodeTypename type, Str name, Opts_def_variable opt )
+CodeVar def_variable( CodeTypename type, Str name, Opts_def_variable p )
 {
-	Opts_def_variable p = get_optional(opt);
-
 	if ( ! name_check( def_variable, name ) || ! null_check( def_variable, type ) ) {
 		GEN_DEBUG_TRAP();
 		return InvalidCode;

base/components/parser.cpp

@@ -1443,22 +1443,6 @@ CodeFn parse_function_after_name(
 	}
 	// <Attributes> <Specifiers> <ReturnType> <Name> ( <Paraemters> ) <Specifiers>
 
-	// Check for trailing specifiers...
-	CodeAttributes post_rt_attributes = parse_attributes();
-	if (post_rt_attributes)
-	{
-		if (attributes)
-		{
-			StrBuilder merged   =  strbuilder_fmt_buf(_ctx->Allocator_Temp, "%S %S", attributes->Content, post_rt_attributes->Content);
-			attributes->Content = cache_str(strbuilder_to_str(merged));
-		}
-		else
-		{
-			attributes = post_rt_attributes;
-		}
-	}
-	// <Attributes> <Specifiers> <ReturnType> <Name> ( <Paraemters> ) <Specifiers> <Attributes> 
-
 	CodeBody    body       = NullCode;
 	CodeComment inline_cmt = NullCode;
 	if ( check( Tok_BraceCurly_Open ) )
@@ -1497,19 +1481,17 @@ CodeFn parse_function_after_name(
 		
 		if ( currtok_noskip.Type == Tok_Comment && currtok_noskip.Line == stmt_end.Line )
 			inline_cmt = parse_comment();
-			// <Attributes> <Specifiers> <ReturnType> <Name> ( <Paraemters> ) <Specifiers> < = 0 or delete > ; <InlineCmt>
+		// <Attributes> <Specifiers> <ReturnType> <Name> ( <Paraemters> ) <Specifiers>; <InlineCmt>
 	}
-
-
-	if (body == nullptr)
+	else
 	{
 		Token stmt_end = currtok;
 		eat( Tok_Statement_End );
-		// <Attributes> <Specifiers> <ReturnType> <Name> ( <Paraemters> ) <Specifiers> <Attributes> < = 0 or delete > ;
+		// <Attributes> <Specifiers> <ReturnType> <Name> ( <Paraemters> ) <Specifiers>;
 
 		if ( currtok_noskip.Type == Tok_Comment && currtok_noskip.Line == stmt_end.Line )
 			inline_cmt = parse_comment();
-			// <Attributes> <Specifiers> <ReturnType> <Name> ( <Paraemters> ) <Specifiers> < = 0 or delete > ; <InlineCmt>
+			// <Attributes> <Specifiers> <ReturnType> <Name> ( <Paraemters> ) <Specifiers>; <InlineCmt>
 	}
 
 	StrBuilder
@@ -1784,7 +1766,7 @@ CodeBody parse_global_nspace( CodeType which )
 
 			case Tok_Preprocess_Macro_Expr: 
 			{
-				if ( ! tok_is_attribute(currtok))
+				if (tok_is_attribute(currtok))
 				{
 					log_failure("Unbounded macro expression residing in class/struct body\n%S", parser_to_strbuilder(_ctx->parser));
 					return InvalidCode;
@@ -2067,20 +2049,11 @@ Token parse_identifier( bool* possible_member_function )
 
 	Macro* macro = lookup_macro(currtok.Text);
 	b32 accept_as_identifier = macro && bitfield_is_set(MacroFlags, macro->Flags, MF_Allow_As_Identifier );
-	b32 is_decarator         = macro && bitfield_is_set(MacroFlags, macro->Flags, MF_Identifier_Decorator );
 
 	// Typename can be: '::' <name>
 	// If that is the case first  option will be Tok_Access_StaticSymbol below
 	if (check(Tok_Identifier) || accept_as_identifier)
-	{
-		if (is_decarator) {
-			Code name_macro = parse_simple_preprocess(currtok.Type);
-			name.Text.Len   = ( ( sptr )prevtok.Text.Ptr + prevtok.Text.Len ) - ( sptr )name.Text.Ptr;
-		}	
-		else {
-			eat(Tok_Identifier);
-		}
-	}
+		eat( Tok_Identifier );
 	// <Name>
 
 	parse_template_args( & name );
@@ -2390,10 +2363,7 @@ CodeOperator parse_operator_after_ret_type(
 		case '(':
 		{
 			if ( currtok.Text.Ptr[1] == ')' )
-			{
 				op = Op_FunctionCall;
-				eat(Tok_Paren_Open);
-			}
 
 			else
 				op = Op_Invalid;
@@ -4744,37 +4714,21 @@ CodeTypename parser_parse_type( bool from_template, bool* typedef_is_function )
 	else if ( currtok.Type == Tok_Decl_Class || currtok.Type == Tok_Decl_Enum || currtok.Type == Tok_Decl_Struct
 			|| currtok.Type == Tok_Decl_Union )
 	{
-		Token next = nexttok;
-
-		if (next.Type == Tok_Identifier)
-		{
-			switch (currtok.Type) {
-				case Tok_Decl_Class  : tag = Tag_Class;  break;
-				case Tok_Decl_Enum   : tag = Tag_Enum;   break;
-				case Tok_Decl_Struct : tag = Tag_Struct; break;
-				case Tok_Decl_Union  : tag = Tag_Union;  break;
-				default:
-					break;
-			}
-			eat( currtok.Type );
-			// <Attributes> <Specifiers> <class, enum, struct, union>
-
-			name = parse_identifier(nullptr);
-			// <Attributes> <Specifiers> <class, enum, struct, union> <Name> 
+		switch (currtok.Type) {
+			case Tok_Decl_Class  : tag = Tag_Class;  break;
+			case Tok_Decl_Enum   : tag = Tag_Enum;   break;
+			case Tok_Decl_Struct : tag = Tag_Struct; break;
+			case Tok_Decl_Union  : tag = Tag_Union;  break;
+			default:
+				break;
 		}
-		else if  (next.Type == Tok_BraceCurly_Open)
-		{
-			name = currtok;
-			// We have an inplace definition, we need to consume that...
+		eat( currtok.Type );
+		// <Attributes> <Specifiers> <class, enum, struct, union>
 
-			// TODO(Ed): we need to add a way for AST_CodeTypename to track an implace definition..
-			b32 const inplace = true;
-			Code indplace_def = cast(Code, parser_parse_struct(inplace));
+		name = parse_identifier(nullptr);
 
-			// For now we lose the structural information, 
-			name.Text.Len = ( ( sptr )prevtok.Text.Ptr + prevtok.Text.Len ) - ( sptr )name.Text.Ptr;
-			// <Attributes> <Specifiers> <class, enum, struct, union> <inplace def>
-		}
+		// name.Length = ( ( sptr )currtok.Text + currtok.Length ) - ( sptr )name.Text;
+		// eat( Tok_Identifier );
 		_ctx->parser.Scope->Name = name.Text;
 		// <Attributes> <Specifiers> <class, enum, struct, union> <Name>
 	}

base/components/parser_types.hpp

@@ -168,36 +168,26 @@ Str macrotype_to_str( MacroType type )
 
 enum EMacroFlags : u16
 {
-	// Macro has parameters (args expected to be passed)
-	MF_Functional          = bit(0), 
-
-	// Expects to assign a braced scope to its body.
-	MF_Expects_Body        = bit(1), 
+	MF_Functional          = bit(0), // Macro has parameters (args expected to be passed)
+	MF_Expects_Body        = bit(1), // Expects to assign a braced scope to its body.
 
 	// lex__eat wil treat this macro as an identifier if the parser attempts to consume it as one.
-	// This is a kludge because we don't support push/pop macro pragmas rn.
+	//  ^^^ This is a kludge because we don't support push/pop macro pragmas rn.
 	MF_Allow_As_Identifier = bit(2), 
 
-	// When parsing identifiers, it will allow the consumption of the macro parameters (as its expected to be a part of constructing the identifier)
-	// Example of a decarator macro from stb_sprintf.h: 
-	// STBSP__PUBLICDEC int STB_SPRINTF_DECORATE(sprintf)(char* buf, char const *fmt, ...) STBSP__ATTRIBUTE_FORMAT(2,3);
-	//                       ^^ STB_SPRINTF_DECORATE is decorating sprintf
-	MF_Identifier_Decorator = bit(3), 
-
 	// lex__eat wil treat this macro as an attribute if the parser attempts to consume it as one.
-	// This a kludge because unreal has a macro that behaves as both a 'statement' and an attribute (UE_DEPRECATED, PRAGMA_ENABLE_DEPRECATION_WARNINGS, etc)
+	//  ^^^ This a kludge because unreal has a macro that behaves as both a 'statement' and an attribute (UE_DEPRECATED, PRAGMA_ENABLE_DEPRECATION_WARNINGS, etc)
 	// TODO(Ed): We can keep the MF_Allow_As_Attribute flag for macros, however, we need to add the ability of AST_Attributes to chain themselves.
 	// Its thats already a thing in the standard language anyway
 	// & it would allow UE_DEPRECATED, (UE_PROPERTY / UE_FUNCTION) to chain themselves as attributes of a resolved member function/variable definition
-	MF_Allow_As_Attribute  = bit(4),
+	MF_Allow_As_Attribute  = bit(3),
 
 	// When a macro is encountered after attributes and specifiers while parsing a function, or variable:
-	// It will consume the macro and treat it as resolving the definition.
+	// It will consume the macro and treat it as resolving the definition. (Yes this is for Unreal Engine)
 	// (MUST BE OF MT_Statement TYPE)
-	MF_Allow_As_Definition = bit(5),
+	MF_Allow_As_Definition = bit(4),
 
-	// Created for Unreal's PURE_VIRTUAL
-	MF_Allow_As_Specifier = bit(6),
+	MF_Allow_As_Specifier = bit(5), // Created for Unreal's PURE_VIRTUAL
 
 	MF_Null           = 0,
 	MF_UnderlyingType = GEN_U16_MAX,

base/dependencies/macros.hpp

@@ -198,16 +198,21 @@
 #ifndef        forceinline
 #	if GEN_COMPILER_MSVC
 #		define forceinline __forceinline
+#		define neverinline __declspec( noinline )
 #	elif GEN_COMPILER_GCC
 #		define forceinline inline __attribute__((__always_inline__))
+#		define neverinline __attribute__( ( __noinline__ ) )
 #	elif GEN_COMPILER_CLANG
 #	if __has_attribute(__always_inline__)
 #		define forceinline inline __attribute__((__always_inline__))
+#		define neverinline __attribute__( ( __noinline__ ) )
 #	else
 #		define forceinline
+#		define neverinline
 #	endif
 #	else
 #		define forceinline
+#		define neverinline
 #	endif
 #endif
 
@@ -314,12 +319,4 @@
 #	define GEN_OPITMIZE_MAPPINGS_END
 #endif
 
-#ifndef get_optional
-#	if GEN_COMPILER_C
-#		define get_optional(opt) opt ? *opt : (typeof(*opt)){0}
-#	else
-#		define get_optional(opt) opt
-#	endif
-#endif
-
 #pragma endregion Macros

base/dependencies/memory.hpp

@@ -138,7 +138,7 @@ constexpr AllocatorInfo heap( void ) { AllocatorInfo allocator = { heap_allocato
 #define malloc( sz ) alloc( heap(), sz )
 
 //! Helper to free memory allocated by heap allocator.
-#define mfree( ptr ) allocator_free( heap(), ptr )
+#define mfree( ptr ) free( heap(), ptr )
 
 struct VirtualMemory
 {

gen_c_library/c_library.cpp

@@ -1159,8 +1159,6 @@ R"(#define <interface_name>( code ) _Generic( (code), \
 				Str actual_name = { fn->Name.Ptr + prefix.Len, fn->Name.Len  - prefix.Len };
 				Str new_name    = StrBuilder::fmt_buf(_ctx->Allocator_Temp, "def__%S", actual_name ).to_str();
 
-				opt_param->ValueType->Specs = def_specifier(Spec_Ptr);
-
 				// Resolve define's arguments
 				b32    has_args   = fn->Params->NumEntries > 1;
 				StrBuilder params_str = StrBuilder::make_reserve(_ctx->Allocator_Temp, 32);
@@ -1174,10 +1172,10 @@ R"(#define <interface_name>( code ) _Generic( (code), \
 				}
 				char const* tmpl_fn_macro = nullptr;
 				if (params_str.length() > 0 ) {
-					tmpl_fn_macro= "#define <def_name>( <params> ... ) <def__name>( <params> & (<opts_type>) { __VA_ARGS__ } )\n";
+					tmpl_fn_macro= "#define <def_name>( <params> ... ) <def__name>( <params> (<opts_type>) { __VA_ARGS__ } )\n";
 				}
 				else {
-					tmpl_fn_macro= "#define <def_name>( ... ) <def__name>( & (<opts_type>) { __VA_ARGS__ } )\n";
+					tmpl_fn_macro= "#define <def_name>( ... ) <def__name>( (<opts_type>) { __VA_ARGS__ } )\n";
 				}
 				Code fn_macro = untyped_str(token_fmt(
 						"def_name",  fn->Name
@@ -1506,7 +1504,6 @@ R"(#define <interface_name>( code ) _Generic( (code), \
 				Str actual_name = { fn->Name.Ptr + prefix.Len, fn->Name.Len  - prefix.Len };
 				Str new_name    = StrBuilder::fmt_buf(_ctx->Allocator_Temp, "def__%S", actual_name ).to_str();
 				fn->Name = cache_str(new_name);
-				opt_param->ValueType->Specs = def_specifier(Spec_Ptr);
 			}
 			src_upfront.append(fn);
 		}

gen_c_library/c_library.refactor

@@ -53,7 +53,6 @@ word enum_underlying,              gen_enum_underlying
 word nullptr,                      gen_nullptr
 word struct_init,                  gen_struct_init
 word hash,                         gen_hash
-word txt,                          gen_txt
 
 // Basic Types
 
@@ -411,8 +410,6 @@ namespace var_,        gen_var_
 
 word _ctx, gen__ctx
 
-word get_context, gen_get_context
-
 word init,   gen_init
 word deinit, gen_deinit
 word reset,  gen_reset

scripts/helpers/vendor_toolchain.ps1

@@ -404,7 +404,7 @@ if ( $vendor -match "msvc" )
 	$flag_optimize_intrinsics		  = '/Oi'
 	$flag_optimized_debug_forceinline = '/d2Obforceinline'
 	$flag_optimized_debug			  = '/Zo'
-	# $flag_
+	$flag_
 	# $flag_out_name                    = '/OUT:'
 	$flag_path_interm                 = '/Fo'
 	$flag_path_debug                  = '/Fd'