FEniCS / ffcx
85%
main: 85%

LAST BUILD BRANCH: gh-readonly-queue/main/pr-877-811fd0a6ce9de0e897f3a9398ac582eab3bc66ea
DEFAULT BRANCH: main
Repo Added
Files 59
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH garth/gate-loop-split-on-mathfunction
branch: garth/gate-loop-split-on-mathfunction
CHANGE BRANCH
x

coverage: 84.928% (+0.03%) from 84.896%
32232180130

Pull #859

github

garth-wells 
Only split the quadrature loop when it has a math function to vectorise

#852 splits the quadrature loop into an evaluation pass and a
contraction pass whenever any fw intermediate exists, so that vectorised
math-library calls (sin, exp, ...) can escape the strided contraction
loop. But an fw intermediate exists for almost every integral -- even a
bare mass matrix has one (the quadrature weight times |detJ|) -- and
when there is nothing to unblock, the split is pure overhead: an extra
loop plus a store/load of every fw through a cache array.

Benchmarked across ten forms on top of this branch (mass matrix, vector
Poisson, elasticity, Taylor-Hood Stokes, N1curl mass, SIPG DG,
HyperElasticity), the unconditional split regresses math-function-free
forms by up to 11-13% at -O2 while leaving the forms #852 targets fully
intact.

Gate the split on the evaluation code actually containing a
MathFunction call, matching the open question raised in #852's own
description. Closes the regression on math-function-free forms (mass
matrix: 0.89x -> 0.99x at -O2) at no cost to the cases #852 targets.
`pytest test/` passes: 46 passed, 1 skipped (unchanged from this
branch).

AI assistance: I used Claude Code (Opus 5) to draft this change,
building on #852. I reviewed and take responsibility for the final
contribution.
Pull Request #859: Only split the quadrature loop when it has a math function to vectorise

23 of 24 new or added lines in 1 file covered. (95.83%)

1 existing line in 1 file now uncovered.

4260 of 5016 relevant lines covered (84.93%)

0.85 hits per line

Relevant lines Covered
5016 RELEVANT LINES 4260 COVERED LINES
0.85 HITS PER LINE
Source Files on garth/gate-loop-split-on-mathfunction

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
32232180130 garth/gate-loop-split-on-mathfunction Only split the quadrature loop when it has a math function to vectorise #852 splits the quadrature loop into an evaluation pass and a contraction pass whenever any fw intermediate exists, so that vectorised math-library calls (sin, exp, ...) can ... Pull #859 garth-wells github
84.93
See All Builds (3196)