Fix allocations due to non-specialized routine

bennibolm · Mar 1, 2024 · 05ed2fd · 05ed2fd
1 parent a3472e5
commit 05ed2fd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/solvers/dgsem_tree/subcell_limiters.jl b/src/solvers/dgsem_tree/subcell_limiters.jl
@@ -96,7 +96,7 @@ function SubcellLimiterIDP(equations::AbstractEquations, basis;
     positivity = (length(positivity_variables_cons) +
                   length(positivity_variables_nonlinear) > 0)
 
-    # When passing `min` or `max` in the elixir the specific function of Base is used.
+    # When passing `min` or `max` in the elixir, the specific function of Base is used.
     # To speed up the simulation, we replace it with `Trixi.min` and `Trixi.max` respectively.
     local_onesided_variables_nonlinear_ = Tuple{Function, Function}[]
     for (variable, min_or_max) in local_onesided_variables_nonlinear
@@ -110,6 +110,7 @@ function SubcellLimiterIDP(equations::AbstractEquations, basis;
             error("Parameter $min_or_max is not a valid input. Use `max` or `min` instead.")
         end
     end
+    local_onesided_variables_nonlinear_ = Tuple(local_onesided_variables_nonlinear_)
 
     local_twosided_variables_cons_ = get_variable_index.(local_twosided_variables_cons,
                                                          equations)

diff --git a/src/solvers/dgsem_tree/subcell_limiters_2d.jl b/src/solvers/dgsem_tree/subcell_limiters_2d.jl
@@ -354,7 +354,7 @@ end
 # Local one-sided limiting of nonlinear variables
 
 @inline function idp_local_onesided!(alpha, limiter, u, t, dt, semi)
-    for (variable, min_or_max) in limiter.local_onesided_variables_nonlinear
+    foreach(limiter.local_onesided_variables_nonlinear) do (variable, min_or_max)
         idp_local_onesided!(alpha, limiter, u, t, dt, semi, variable, min_or_max)
     end