Fix a couple of mismatching argument comments.

PiperOrigin-RevId: 721529013

Author: hzeller

xls/codegen/proc_block_conversion.cc

@@ -486,7 +486,7 @@ static absl::StatusOr<std::vector<Node*>> MakeValidNodesForInputStates(
           Node * state_valid,
           NaryOrIfNeeded(block, state_valid_conditions,
                          /*name=*/state_valid_name,
-                         /*loc=*/state_register->reg_full_read->loc()));
+                         /*source_info=*/state_register->reg_full_read->loc()));
       active_valids.push_back(state_valid);
     }
 

xls/dslx/bytecode/bytecode_interpreter_test.cc

@@ -120,7 +120,7 @@ static absl::StatusOr<InterpValue> Interpret(
   XLS_RET_CHECK_EQ(bf->owner(), f->owner());
 
   return BytecodeInterpreter::Interpret(import_data, bf.get(), args,
-                                        /*hierarchy_interpreter=*/std::nullopt,
+                                        /*channel_manager=*/std::nullopt,
                                         options);
 }
 
@@ -138,10 +138,10 @@ TEST_F(BytecodeInterpreterTest, DupLiteral) {
                                /*type_info=*/nullptr, std::move(bytecodes)));
   BytecodeInterpreterOptions options;
   options.set_validate_final_stack_depth(false);
-  XLS_ASSERT_OK_AND_ASSIGN(
-      InterpValue result, BytecodeInterpreter::Interpret(
-                              &import_data_.value(), bfunc.get(), {},
-                              /*hierarchy_interpreter=*/std::nullopt, options));
+  XLS_ASSERT_OK_AND_ASSIGN(InterpValue result,
+                           BytecodeInterpreter::Interpret(
+                               &import_data_.value(), bfunc.get(), {},
+                               /*channel_manager=*/std::nullopt, options));
   EXPECT_EQ(result.ToString(), "u32:42");
 }
 

xls/dslx/frontend/ast_utils.cc

@@ -316,7 +316,7 @@ static std::optional<bool> TryEvaluateAsBool(const Expr* expr) {
         GetSignAndMagnitude(number->text());
     if (sm.ok() && sm->second.bit_count() <= 1) {
       // Note: the zero value is given as a bit count of zero.
-      bool value = sm->second.bit_count() == 0 ? 0 : sm->second.Get(0);
+      bool value = sm->second.bit_count() == 0 ? false : sm->second.Get(0);
       return std::optional<bool>(value);
     }
   }

xls/dslx/interp_value_utils_test.cc

@@ -270,8 +270,8 @@ TEST(InterpValueHelpersTest, ValueToInterpValue) {
 
 TEST(InterpValueHelpersTest, ValueToInterpValueEnum) {
   EnumDef enum_def(/*owner=*/nullptr, /*span=*/Span::Fake(),
-                   /*name_def=*/nullptr, /*parametric_bindings=*/{},
-                   /*members=*/{}, /*is_public=*/false);
+                   /*name_def=*/nullptr, /*type=*/{},
+                   /*values=*/{}, /*is_public=*/false);
   EnumType enum_type(enum_def, TypeDim::CreateU32(32), /*is_signed=*/false, {});
   EXPECT_THAT(ValueToInterpValue(Value(UBits(3, 32)), &enum_type),
               IsOkAndHolds(Eq(InterpValue::MakeEnum(

xls/dslx/run_routines/run_routines.cc

@@ -146,7 +146,7 @@ absl::Status RunDslxTestFunction(ImportData* import_data, TypeInfo* type_info,
           BytecodeEmitterOptions{.format_preference =
                                      options.format_preference()}));
   return BytecodeInterpreter::Interpret(import_data, bf.get(), /*args=*/{},
-                                        /*hierarchy_interpreter=*/std::nullopt,
+                                        /*channel_manager=*/std::nullopt,
                                         options)
       .status();
 }

xls/ir/package_test.cc

@@ -333,7 +333,7 @@ TEST_F(PackageTest, CreateStreamingChannel) {
       Channel * ch42,
       p.CreateStreamingChannel(
           "ch42", ChannelOps::kReceiveOnly, p.GetBitsType(44),
-          /*initial_values=*/{}, /*channel_config=*/std::nullopt,
+          /*initial_values=*/{}, /*fifo_config=*/std::nullopt,
           FlowControl::kReadyValid, ChannelStrictness::kProvenMutuallyExclusive,
           ChannelMetadataProto(), 42));
   EXPECT_EQ(ch42->id(), 42);
@@ -348,7 +348,7 @@ TEST_F(PackageTest, CreateStreamingChannel) {
   // Creating a channel with a duplicate ID is an error.
   EXPECT_THAT(p.CreateStreamingChannel(
                    "ch1_dup", ChannelOps::kReceiveOnly, p.GetBitsType(44),
-                   /*initial_values=*/{}, /*channel_config=*/std::nullopt,
+                   /*initial_values=*/{}, /*fifo_config=*/std::nullopt,
                    FlowControl::kReadyValid,
                    ChannelStrictness::kProvenMutuallyExclusive,
                    ChannelMetadataProto(), 1)

xls/passes/conditional_specialization_pass_test.cc

@@ -1219,7 +1219,7 @@ TEST_F(ConditionalSpecializationPassTest, StateReadSpecialization) {
   BValue counter0 = pb.StateElement("counter0", UBits(0, 32));
   BValue counter1 = pb.StateElement("counter1", UBits(5, 32));
   BValue selected_counter =
-      pb.Select(index_is_1, {counter1}, /*default_value=*/counter0);
+      pb.Select(index_is_1, {counter1}, /*on_false=*/counter0);
   BValue incremented_counter =
       pb.Add(selected_counter, pb.Literal(UBits(1, 32)));
   pb.Next(index, index_is_0);
@@ -1252,7 +1252,7 @@ TEST_F(ConditionalSpecializationPassTest, StateReadSpecializationDisabled) {
   BValue counter0 = pb.StateElement("counter0", UBits(0, 32));
   BValue counter1 = pb.StateElement("counter1", UBits(5, 32));
   BValue selected_counter =
-      pb.Select(index_is_1, {counter1}, /*default_value=*/counter0);
+      pb.Select(index_is_1, {counter1}, /*on_false=*/counter0);
   BValue incremented_counter =
       pb.Add(selected_counter, pb.Literal(UBits(1, 32)));
   pb.Next(index, index_is_0);