Skip to content

compiler: Fix performance profiling of busy-waits - #3055

Open
FabioLuporini wants to merge 3 commits into
mainfrom
fix-wait-timing
Open

FabioLuporini wants to merge 3 commits into
mainfrom
fix-wait-timing

Conversation

@FabioLuporini

Copy link
Copy Markdown
Contributor

No description provided.

@FabioLuporini FabioLuporini self-assigned this Oct 14, 2026
@FabioLuporini FabioLuporini added the bug-C-minor bug in the generated code not affecting correctness label Oct 14, 2026
if size == 1:
continue
if v.is_double_buffering:
# E.g., `f[time] = b[t0]` must become `b1[t0] = b[t0]`

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

actually this is a bug fix (tested both here and in PRO)

@codecov

codecov Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.15385% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.32%. Comparing base (2707ac9) to head (ea8fc74).

Files with missing lines Patch % Lines
devito/passes/clusters/buffering.py 55.55% 3 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3055      +/-   ##
==========================================
+ Coverage   84.23%   84.32%   +0.09%     
==========================================
  Files         258      258              
  Lines       57070    57151      +81     
  Branches     4874     4885      +11     
==========================================
+ Hits        48072    48192     +120     
+ Misses       8170     8133      -37     
+ Partials      828      826       -2     
Flag Coverage Δ
pytest-gpu-aomp-amdgpuX 68.49% <42.85%> (-0.03%) ⬇️
pytest-gpu-gcc- 79.00% <47.11%> (-0.01%) ⬇️
pytest-gpu-icx- 78.94% <47.11%> (+0.02%) ⬆️
pytest-gpu-nvc-nvidiaX 69.14% <82.89%> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Select compute_100a/sm_100a in CudaCompiler for detected SM100 devices.
Use explicit Blackwell as the SM100 fallback when device detection is
unavailable, and preserve other detected targets.

Explicit gencode avoids the generic compute_100 PTX fallback generated by
-arch=sm_100a, which rejects architecture-specific instructions such as
tcgen05. Target selection no longer needs a downstream pass override.

Add hardware-independent coverage of native fallback, older GPUs, SM100,
and other Blackwell targets.
Reuse the first buffer's slot indices when inserting further buffering
levels, including initialization outside the time loop. Preserve physical
irregular-buffer slots in copy-in masks and flatten transfers using their
allocated stride independently of the transferred payload size.

Add a subsampled multilevel buffering regression for degrees 1, 2 and 4.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-C-minor bug in the generated code not affecting correctness

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants