should remove redundant function first, then clean callerMap

Addresses #653.
phillipstanleymarbell · May 22, 2023 · cf7e76b · cf7e76b
1 parent 5693026
commit cf7e76b
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/src/newton/newton-irPass-LLVMIR-optimizeByRange.cpp b/src/newton/newton-irPass-LLVMIR-optimizeByRange.cpp
@@ -361,10 +361,10 @@ irPassLLVMIROptimizeByRange(State * N, bool enableOverload, bool enableBuiltinAs
 	 * remove the functions that are optimized by passes.
 	 * */
     if (useOverLoad)
-        cleanFunctionMap(Mod, callerMap);
+        overloadFunc(Mod, callerMap);
 
     if (useOverLoad)
-        overloadFunc(Mod, callerMap);
+        cleanFunctionMap(Mod, callerMap);
 
     callerMap.clear();
     funcBoundInfo.clear();
@@ -405,11 +405,10 @@ irPassLLVMIROptimizeByRange(State * N, bool enableOverload, bool enableBuiltinAs
     /*
      * remove the functions that are optimized by passes.
      * */
-    if (useOverLoad)
-        cleanFunctionMap(Mod, callerMap);
-
     if (useOverLoad)
         overloadFunc(Mod, callerMap);
+    if (useOverLoad)
+        cleanFunctionMap(Mod, callerMap);
 
     flexprint(N->Fe, N->Fm, N->Fpinfo, "infer bound\n");
     callerMap.clear();
@@ -442,11 +441,10 @@ irPassLLVMIROptimizeByRange(State * N, bool enableOverload, bool enableBuiltinAs
     /*
 	 * remove the functions that are optimized by passes.
 	 * */
-    if (useOverLoad)
-        cleanFunctionMap(Mod, callerMap);
-
     if (useOverLoad)
         overloadFunc(Mod, callerMap);
+    if (useOverLoad)
+        cleanFunctionMap(Mod, callerMap);
 
     if (enableBuiltinAssume) {
         for (auto & mi : *Mod)