Commit 27f3e8b61db897f36c704c50c86c31b824297d30 | zig

author	Matthew Lugg <mlugg@mlugg.co.uk> 2025-03-26 02:52:48+00:00
committer	Alex Rønne Petersen <alex@alexrp.com> 2025-03-26 15:41:09+01:00
log	27f3e8b61db897f36c704c50c86c31b824297d30
tree	9e72489ed38d6262f194203ab49ea528b4525e12
parent	9c857bb32d838a14fdf9fd09d88d74bb9ccfc72e
signature	Signed by SSH key SHA256:7B/LJ7bpR1eX8aCXSr4mtd5M45VMPKcx9zY8e95b5QM

Zcu: include named tests in resolved references

Oops, a little typo from yours truly! No test for this one, because we
don't have any way of testing the reference trace.

1 files changed, 9 insertions(+), 4 deletions(-)

src
- Zcu.zig

src/Zcu.zig+9-4

 @@ -3872,9 +3872,11 @@ fn resolveReferencesInner(zcu: *Zcu) !std.AutoHashMapUnmanaged(AnalUnit, ?Resolv
                     .unnamed_test => true,
                     .@"test", .decltest => a: {
                         const fqn_slice = nav.fqn.toSlice(ip);
-                        for (comp.test_filters) |test_filter| {
+                        if (comp.test_filters.len > 0) {
-                            if (std.mem.indexOf(u8, fqn_slice, test_filter) != null) break;
+                            for (comp.test_filters) |test_filter| {
-                        } else break :a false;
+                                if (std.mem.indexOf(u8, fqn_slice, test_filter) != null) break;
+                            } else break :a false;
+                        }
                         break :a true;
                     },
                 };
 @@ -3884,7 +3886,10 @@ fn resolveReferencesInner(zcu: *Zcu) !std.AutoHashMapUnmanaged(AnalUnit, ?Resolv
                         @intFromEnum(inst_info.inst),
                     });
                     try unit_queue.put(gpa, .wrap(.{ .nav_val = nav_id }), referencer);
-                    try unit_queue.put(gpa, .wrap(.{ .func = nav.status.fully_resolved.val }), referencer);
+                    // Non-fatal AstGen errors could mean this test decl failed
+                    if (nav.status == .fully_resolved) {
+                        try unit_queue.put(gpa, .wrap(.{ .func = nav.status.fully_resolved.val }), referencer);
+                    }
                 }
             }
             for (zcu.namespacePtr(ns).pub_decls.keys()) |nav| {