From 3b75be08edad89c79692b7f5a58654ee4acc734c Mon Sep 17 00:00:00 2001 From: Adam Siemieniuk Date: Fri, 20 Mar 2026 16:29:49 +0100 Subject: [PATCH] [schedule] Better named sequence builder result default Switches named sequeunce builder to expect no results by default. This aligns better with most schedules that do not return any values. --- lighthouse/schedule/builders.py | 4 +--- lighthouse/workload/runner.py | 5 ++++- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/lighthouse/schedule/builders.py b/lighthouse/schedule/builders.py index 70d8351..787e327 100644 --- a/lighthouse/schedule/builders.py +++ b/lighthouse/schedule/builders.py @@ -31,7 +31,7 @@ def create_named_sequence( Args: sym_name: Sequence name input_types: Input types (default: a single arg) - result_types: Result types (default: a single return) + result_types: Result types (default: no returns) is_readonly: Mark inputs as readonly Returns: @@ -40,8 +40,6 @@ def create_named_sequence( if not input_types: input_types = [transform.any_op_t()] - if not result_types: - result_types = [transform.any_op_t()] arg_attrs = [{"transform.consumed": ir.UnitAttr.get()}] if is_readonly: diff --git a/lighthouse/workload/runner.py b/lighthouse/workload/runner.py index c3d3017..673abc6 100644 --- a/lighthouse/workload/runner.py +++ b/lighthouse/workload/runner.py @@ -70,7 +70,10 @@ def execute( def get_bench_wrapper_schedule(workload: Workload): - with schedule_boilerplate() as (schedule, named_seq): + with schedule_boilerplate(result_types=[transform.any_op_t()]) as ( + schedule, + named_seq, + ): named_func = structured.structured_match( transform.AnyOpType.get(), target=named_seq.bodyTarget,