diff lwlink/link.c @ 155:1571e150f1fd

Fixed error where external references within a library file were not resolved when linking
author lost@l-w.ca
date Sun, 28 Aug 2011 00:07:15 -0600
parents 729a9c70934e
children fc8386b13399
line wrap: on
line diff
--- a/lwlink/link.c	Sat Aug 27 23:16:50 2011 -0600
+++ b/lwlink/link.c	Sun Aug 28 00:07:15 2011 -0600
@@ -213,8 +213,10 @@
 		{
 			if (!strcmp(sym, (char *)(se -> sym)))
 			{
+//				fprintf(stderr, "Found symbol %s in %s\n", sym, fn -> filename);
 				if (!(fn -> forced))
 				{
+//					fprintf(stderr, "   Forced\n");
 					fn -> forced = 1;
 					nforced = 1;
 				}
@@ -233,8 +235,10 @@
 		r = find_external_sym_recurse(sym, fn -> subs[sn]);
 		if (r)
 		{
+//			fprintf(stderr, "Found symbol %s in %s\n", sym, fn -> filename);
 			if (!(fn -> forced))
 			{
+//				fprintf(stderr, "    Forced\n");
 				nforced = 1;
 				fn -> forced = 1;
 			}
@@ -256,6 +260,8 @@
 	symtab_t *se;
 	fileinfo_t *fp;
 
+//	fprintf(stderr, "Looking up %s\n", sym);
+
 	if (symtype == 1)
 	{
 		// local symbol
@@ -397,17 +403,47 @@
 		check_os9();	
 }
 
+void resolve_files_aux(fileinfo_t *fn)
+{
+	int sn;
+	reloc_t *rl;
+	lw_expr_stack_t *te;
+	
+	int rval;
+
+	
+	if (fn -> forced == 0)
+		return;
+	
+	for (sn = 0; sn < fn -> nsections; sn++)
+	{
+		for (rl = fn -> sections[sn].incompletes; rl; rl = rl -> next)
+		{
+			// do a "simplify" on the expression
+			te = lw_expr_stack_dup(rl -> expr);
+			rval = lw_expr_reval(te, resolve_sym, &(fn -> sections[sn]));
+			
+			// is it constant? error out if not
+			if (rval != 0 || !lw_expr_is_constant(te))
+			{
+				fprintf(stderr, "Incomplete reference at %s:%s+%02X\n", fn -> filename, fn -> sections[sn].name, rl -> offset);
+				symerr = 1;
+			}
+			lw_expr_stack_free(te);
+		}
+	}
+
+	// handle any sub files
+	for (sn = 0; sn < fn -> nsubs; sn++)
+		resolve_files_aux(fn -> subs[sn]);
+}
+
 /*
 This is just a pared down version of the algo for resolving references.
 */
 void resolve_files(void)
 {
-	int sn;
 	int fn;
-	reloc_t *rl;
-	lw_expr_stack_t *te;
-	
-	int rval;
 
 	// resolve entry point if required
 	// this must resolve to an *exported* symbol and will resolve to the
@@ -429,26 +465,7 @@
 		nforced = 0;
 		for (fn = 0; fn < ninputfiles; fn++)
 		{
-			if (inputfiles[fn] -> forced == 0)
-				continue;
-	
-			for (sn = 0; sn < inputfiles[fn] -> nsections; sn++)
-			{
-				for (rl = inputfiles[fn] -> sections[sn].incompletes; rl; rl = rl -> next)
-				{
-					// do a "simplify" on the expression
-					te = lw_expr_stack_dup(rl -> expr);
-					rval = lw_expr_reval(te, resolve_sym, &(inputfiles[fn] -> sections[sn]));
-					
-					// is it constant? error out if not
-					if (rval != 0 || !lw_expr_is_constant(te))
-					{
-						fprintf(stderr, "Incomplete reference at %s:%s+%02X\n", inputfiles[fn] -> filename, inputfiles[fn] -> sections[sn].name, rl -> offset);
-						symerr = 1;
-					}
-					lw_expr_stack_free(te);
-				}
-			}
+			resolve_files_aux(inputfiles[fn]);
 		}
 	}
 	while (nforced == 1);