Skip to content

Fix memory leak for direction_dependent BCs#147

Merged
cgilet merged 2 commits intoAMReX-Fluids:developmentfrom
cgilet:fix_memory_leak
Jun 2, 2025
Merged

Fix memory leak for direction_dependent BCs#147
cgilet merged 2 commits intoAMReX-Fluids:developmentfrom
cgilet:fix_memory_leak

Conversation

@cgilet
Copy link
Copy Markdown
Collaborator

@cgilet cgilet commented Jun 2, 2025

No description provided.

@cgilet cgilet requested a review from asalmgren June 2, 2025 14:50
@asalmgren asalmgren requested a review from WeiqunZhang June 2, 2025 18:02
Copy link
Copy Markdown
Member

@WeiqunZhang WeiqunZhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could also do

Vector<Array<MultiFab,AMREX_SPACEDIM>> vel_vec(finest_level+1);

for (int lev = 0; lev <= finest_level; lev++) {
    auto& ld = *m_leveldata[lev];
    AMREX_D_TERM(vel_vec[lev][0] = MultiFab(ld.velocity, amrex::make_alias, 0, 1);,
                 vel_vec[lev][1] = MultiFab(ld.velocity, amrex::make_alias, 1, 1);,
                 vel_vec[lev][2] = MultiFab(ld.velocity, amrex::make_alias, 2, 1););
}

HydroUtils::enforceInOutSolvability(GetVecOfArrOfPtrs(vel_vec), get_velocity_bcrec().data(), geom, true);

@cgilet cgilet merged commit 5104d13 into AMReX-Fluids:development Jun 2, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants