gmarkall/manycore_form_compiler — Issue #56: Multiplying form expressions by detJ renders Partitioner obsolete
Author: kynan
Created: 2012-01-19 17:14:38+00:00
State: open

In its current form, the `Partitioner` only acts on sums if they're at the root of the expression tree and splits the expression into partitions. When multiplying the entire form expression by `Determinant(J)`, the root of the tree is always a product and hence the `Partitioner` will always only find a single partition.

When `detJ` was evaluated as a common subexpression before the loop, the `Partitioner` could sensibly act on the remaining expression and multiplying the common subexpressions to each partition.