This diff contains no algorithmic changes.

Estimated hours taken: 6
Branches: main

This diff contains no algorithmic changes. It merely renames apart a bunch more
function symbols to reduce ambiguity.

After this diff, the summary line from the mdb command "ambiguity -f" is

	Total: 351 names used 975 times, maximum 31, average: 2.78

browser/*.m:
compiler/*.m:
	Rename function symbols to eliminate ambiguities.

tests/debugger/declarative/dependency.exp:
tests/debugger/declarative/dependency2.exp:
	Update the expected out where some internal function symbol names
	appear in the output of the debugger. (This output is meant for
	implementors only.)

compiler/ml_optimize.m

@@ -257,8 +257,9 @@ optimize_in_call_stmt(OptInfo, Stmt0) = Stmt :-
         % the --target asm back-end, whereas generating the
         % appropriate MLDS instructions does.
         %
-        FuncRval = const(code_addr_const(
-            proc(qual(ModName, module_qual, ProcLabel), _FuncSignature))),
+        FuncRval = const(mlconst_code_addr(
+            code_addr_proc(qual(ModName, module_qual, ProcLabel),
+                _FuncSignature))),
         ProcLabel = mlds_proc_label(PredLabel, _ProcId),
         PredLabel = mlds_user_pred_label(predicate, _DefnModName, PredName,
             _Arity, _CodeModel, _NonOutputFunc),
@@ -418,7 +419,7 @@ optimize_func_stmt(OptInfo, statement(Stmt0, Context)) =
             %   }
             % Any tail calls in the function body will have
             % been replaced with `continue' statements.
-            Stmt = while(const(true_const),
+            Stmt = while(const(mlconst_true),
                 statement(block([],
                     [CommentStmt,
                     statement(Stmt0, Context),