Merge branch 'main' of git.nbross.com:nielsbross/Ink-Shader-Language

Check.jai

@@ -6,6 +6,8 @@
 //~ nbr: Error reporting TODOs
 //
 // [ ] Add and error for using keywords as names, or rename the dx11 keywords in the resulting hlsl shader.
+// [ ] Add missing scope in for loop for loop iterator
+// [ ] Maybe we remove the resource index on type variables, seems like we don't need it there.
 // [x] Improve error reporting on mismatched overloads when types don't match, but arity does.
 // [x] Improve error reporting for type mismatches in general. It seems like the expect node is not always correct.
 
@@ -83,6 +85,10 @@ Type_Variable :: struct {
 	MAX_TYPE_VARIABLE_CHILDREN :: 32;
 	children : Static_Array(Type_Variable_Handle, MAX_TYPE_VARIABLE_CHILDREN);
 
+	//@Incomplete: Should we remove this and do it in the output generation instead?
+	//             Seems kind of superfluous considering we auto-generate it.
+	//             Otherwise if we want to specify it, it should be specified as a hint @binding(index)
+	//             In the case of using hints, we can just have it on the AST node.
 	//@Note(niels): For constant buffers
 	resource_index : u32;
 
@@ -1639,6 +1645,7 @@ check_node :: (checker : *Checker, node : *AST_Node) -> Type_Variable_Handle {
 		}
 		case .For; {
 			loop_iterator := node.token;
+			// @Incomplete: Missing scope here?
 
 			scope, scope_handle := push_scope(checker, kind = .Block);