Update HandleProcNextFunction to accept a ConversionRecord instead of the individual members of the record.

This will allow us to pass in additional information via the ConversionRecord without expanding the number of parameters to that method.

PiperOrigin-RevId: 808641832

Author: dplassgit

xls/dslx/ir_convert/BUILD

@@ -328,6 +328,7 @@ cc_library(
     deps = [
         ":channel_scope",
         ":conversion_info",
+        ":conversion_record",
         ":convert_format_macro",
         ":convert_options",
         ":ir_conversion_utils",

xls/dslx/ir_convert/function_converter.cc

@@ -54,6 +54,7 @@
 #include "xls/dslx/interp_value.h"
 #include "xls/dslx/interp_value_utils.h"
 #include "xls/dslx/ir_convert/channel_scope.h"
+#include "xls/dslx/ir_convert/conversion_record.h"
 #include "xls/dslx/ir_convert/convert_format_macro.h"
 #include "xls/dslx/ir_convert/convert_options.h"
 #include "xls/dslx/ir_convert/ir_conversion_utils.h"
@@ -3275,16 +3276,21 @@ absl::Status FunctionConverter::HandleChannelDecl(const ChannelDecl* node) {
 
 // TODO: davidplass - break this method up. It's too big.
 absl::Status FunctionConverter::HandleProcNextFunction(
-    Function* f, const Invocation* invocation, TypeInfo* type_info,
-    ImportData* import_data, const ParametricEnv* parametric_env,
-    const ProcId& proc_id, ProcConversionData* proc_data) {
+    const ConversionRecord& record, ImportData* import_data,
+    ProcConversionData* proc_data) {
+  TypeInfo* type_info = record.type_info();
   XLS_RET_CHECK_NE(type_info, nullptr);
+
+  Function* f = record.f();
+  ProcId proc_id = record.proc_id().value();
   VLOG(5) << "HandleProcNextFunction: " << f->ToString() << " proc id "
           << proc_id.ToString();
+  const Invocation* invocation = record.invocation();
   if (invocation != nullptr) {
     VLOG(5) << "HandleProcNextFunction: invocation " << invocation->ToString();
   }
 
+  const ParametricEnv* parametric_env = &record.parametric_env();
   if (parametric_env != nullptr) {
     SetParametricEnv(parametric_env);
   }

xls/dslx/ir_convert/function_converter.h

@@ -38,6 +38,7 @@
 #include "xls/dslx/interp_value.h"
 #include "xls/dslx/ir_convert/channel_scope.h"
 #include "xls/dslx/ir_convert/conversion_info.h"
+#include "xls/dslx/ir_convert/conversion_record.h"
 #include "xls/dslx/ir_convert/convert_options.h"
 #include "xls/dslx/ir_convert/proc_config_ir_converter.h"
 #include "xls/dslx/ir_convert/proc_scoped_channel_scope.h"
@@ -149,10 +150,9 @@ class FunctionConverter {
   absl::Status HandleFunction(Function* node, TypeInfo* type_info,
                               const ParametricEnv* parametric_env);
 
-  absl::Status HandleProcNextFunction(
-      Function* pf, const Invocation* invocation, TypeInfo* type_info,
-      ImportData* import_data, const ParametricEnv* parametric_env,
-      const ProcId& proc_id, ProcConversionData* proc_data);
+  absl::Status HandleProcNextFunction(const ConversionRecord& record,
+                                      ImportData* import_data,
+                                      ProcConversionData* proc_data);
 
   // Notes a constant-definition dependency for the function (so it can
   // participate in the IR conversion).

xls/dslx/ir_convert/ir_converter.cc

@@ -226,9 +226,7 @@ absl::Status ConvertOneFunctionInternal(PackageData& package_data,
       proc_data->id_to_initial_value[record.proc_id().value()] = ir_value;
     }
 
-    return converter.HandleProcNextFunction(
-        f, record.invocation(), record.type_info(), import_data,
-        &record.parametric_env(), record.proc_id().value(), proc_data);
+    return converter.HandleProcNextFunction(record, import_data, proc_data);
   }
 
   return converter.HandleFunction(f, record.type_info(),