Skip to content

Conversation

christiangnrd
Copy link
Contributor

@christiangnrd christiangnrd commented Oct 13, 2025

Probably missed because when there's a broken, failing, or erroring test this bug is hidden.

Before (bug is on Test Summary line):

                                 │          │ ──────────────── CPU ──────────────── │
Test                    (Worker) │ Time (s) │ GC (s) │ GC % │ Alloc (MB) │ RSS (MB) │
device/intrinsics/arguments  (3) │    14.06 │   0.33 │  2.4 │    2553.03 │   804.14 │
device/intrinsics/atomics    (2) │    20.33 │   0.44 │  2.2 │    4131.85 │   846.61 │

Test Summary: | Pass  Total   TimeA: ~0 min
  Overall     |  118    118  26.4s
    SUCCESS
     Testing Metal tests passed

This PR:

                                 │          │ ──────────────── CPU ──────────────── │
Test                    (Worker) │ Time (s) │ GC (s) │ GC % │ Alloc (MB) │ RSS (MB) │
device/intrinsics/arguments  (3) │    14.03 │   0.33 │  2.3 │    2552.47 │   806.64 │
device/intrinsics/atomics    (2) │    20.65 │   0.46 │  2.2 │    4132.25 │   852.47 │

Test Summary: | Pass  Total   Time
  Overall     |  118    118  27.4s
    SUCCESS
     Testing Metal tests passed

Includes a bonus unused variable cleanup.

@maleadt maleadt merged commit d2a01f1 into JuliaTesting:main Oct 14, 2025
20 checks passed
@christiangnrd christiangnrd deleted the fixclear branch October 14, 2025 10:13
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