Skip to content

Remove dedicated LIR for_loop node#9299

Draft
rtfeldman wants to merge 1232 commits intomainfrom
codex/remove-lir-for-loop
Draft

Remove dedicated LIR for_loop node#9299
rtfeldman wants to merge 1232 commits intomainfrom
codex/remove-lir-for-loop

Conversation

@rtfeldman
Copy link
Copy Markdown
Contributor

@rtfeldman rtfeldman commented Mar 27, 2026

No description provided.

Comment thread src/eval/cir_to_lir.zig Outdated
Comment on lines -23 to -32
/// Comptime-gated tracing for the shared lowering pipeline.
const trace = struct {
const enabled = if (@hasDecl(build_options, "trace_eval")) build_options.trace_eval else false;

fn log(comptime fmt: []const u8, args: anytype) void {
if (comptime enabled) {
std.debug.print("[lower] " ++ fmt ++ "\n", args);
}
}
};
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found this tracing helpful for tracking down bugs, do we want to remove it?

@rtfeldman rtfeldman force-pushed the codex/remove-lir-for-loop branch 2 times, most recently from 43b6417 to e5bb427 Compare April 19, 2026 01:12
rtfeldman added 30 commits May 4, 2026 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants