From 20d138e3a300f2c334ba112ccb41927f7ba7d570 Mon Sep 17 00:00:00 2001
From: Arne Morten Kvarving <arne.morten.kvarving@sintef.no>
Date: Fri, 21 Feb 2025 11:15:55 +0100
Subject: [PATCH] move block data extraction into shared loop

no reason to setup the element contexts twice
---
 opm/simulators/flow/EclWriter.hpp | 11 +----------
 1 file changed, 1 insertion(+), 10 deletions(-)

diff --git a/opm/simulators/flow/EclWriter.hpp b/opm/simulators/flow/EclWriter.hpp
index f742bb0a5e..a2fff5ccc6 100644
--- a/opm/simulators/flow/EclWriter.hpp
+++ b/opm/simulators/flow/EclWriter.hpp
@@ -731,22 +731,13 @@ private:
                 elemCtx.updatePrimaryIntensiveQuantities(/*timeIdx=*/0);
 
                 this->outputModule_->processElement(elemCtx);
+                this->outputModule_->processElementBlockData(elemCtx);
             }
             this->outputModule_->clearExtractors();
 
             this->outputModule_->accumulateDensityParallel();
         }
 
-        {
-            OPM_TIMEBLOCK(prepareBlockData);
-            for (const auto& elem : elements(gridView, Dune::Partitions::interior)) {
-                elemCtx.updatePrimaryStencil(elem);
-                elemCtx.updatePrimaryIntensiveQuantities(/*timeIdx=*/0);
-
-                this->outputModule_->processElementBlockData(elemCtx);
-            }
-        }
-
         {
             OPM_TIMEBLOCK(prepareFluidInPlace);