ffc team mailing list archive

["Garth N. Wells" <g.n.wells@xxxxxxxxxx>]

Anders,

Attached is patch file that gives a total speed up of about 30% for the
forms I've been testing. It's just a rearrangement of a loop in one
function in elementtensor.py. 

Garth

diff -r d48e31a478d9 src/ffc/compiler/elementtensor.py
--- a/src/ffc/compiler/elementtensor.py	Mon May  8 15:05:00 2006 +0200
+++ b/src/ffc/compiler/elementtensor.py	Wed May 10 17:18:13 2006 +0200
@@ -134,21 +134,23 @@ class ElementTensor:
         debug("Generating code for element tensor", 1)         
         declarations = []
         iindices = self.terms[0].A0.i.indices or [[]] # All primary ranks are equal
+
+        # Compute geometry tensor 
+        gk_tensor = [ ( [(format.format["geometry tensor"](j, a), a) for a in self.__aindices(j) ], j) \
+              for j in range(len(self.terms)) ]
+
         k = 0 # Update counter for each entry of A0, which is needed for some formats
         num_dropped = 0
         for i in iindices:
             debug("i = " + str(i), 2)
             value = None
-            for j in range(len(self.terms)):
+            name = format.format["element tensor"](i, k)
+            for (gka, j) in gk_tensor:
                 debug("  j = " + str(j), 2)
                 A0 = self.terms[j].A0
-                if A0.a.indices: aindices = A0.a.indices
-                else: aindices = [[]]
-                for a in aindices:
+                for (gk, a) in gka:
                     debug("    a = " + str(a), 2)
-                    name = format.format["element tensor"](i, k)
                     a0 = A0(i, a)
-                    gk = format.format["geometry tensor"](j, a)
                     debug("      a0 = " + str(a0), 2)
                     debug("      gk = " + str(gk), 2)
                     if abs(a0) > FFC_EPSILON:
@@ -195,3 +197,10 @@ class ElementTensor:
         for p in sum.products:
             if not p.integral:
                 raise FormError, (p, "Missing integral in term.")
+
+    def __aindices(self, j):
+        A0 = self.terms[j].A0
+        if A0.a.indices: aindices = A0.a.indices
+        else: aindices = [[]]
+        return aindices
+