Skip to content

enable COMP=clang for TPL#434

Merged
baperry2 merged 2 commits intodevelopmentfrom
tpl-clang
Oct 26, 2023
Merged

enable COMP=clang for TPL#434
baperry2 merged 2 commits intodevelopmentfrom
tpl-clang

Conversation

@baperry2
Copy link
Copy Markdown
Contributor

The AMReX GNUmake system interprets COMP=llvm and COMP=clang equivalently, so we should too for TPL.

@baperry2 baperry2 requested a review from jrood-nrel October 25, 2023 22:50
@jrood-nrel
Copy link
Copy Markdown
Contributor

While you're at it, can you change intel to intel-classic?

@baperry2 baperry2 enabled auto-merge (squash) October 26, 2023 19:29
@baperry2 baperry2 merged commit 6d14b08 into development Oct 26, 2023
@baperry2 baperry2 deleted the tpl-clang branch October 26, 2023 19:32
baperry2 added a commit that referenced this pull request Dec 1, 2023
* Explicit filtering of hydro fluxes and source term. (#112)

This commit implements explicit filtering of the hydro fluxes and
source term (for both MOL and PPM code paths). This is necessary to
perform explicitly filtered Large Eddy Simulations.

Contains:
- regressions tests
- documentation
- CMake implementation

* Remove legacy Fortran code. (#252)

* Use default for trivial constructors and only build HIP in CI for gfx908 (#434)

* Use default for trivial constructors.

* Only build HIP in CI for gfx908.

* Update PelePhysics.

* Fixes to header files (#439)

* Fixes to header files.

* Update PelePhysics submodule.

* More fixes.

* Initialize xdiff in HIT.

* Invert some mfiter loops (#446)

Co-authored-by: Jon Rood <[email protected]>

* Add GPU syncs because MF ParallelFor is non-blocking (#471)

* Rename header guards (#506)

* Replace _OPENMP with AMREX_USE_OMP to support Ascent. (#519)

* Address some warnings (#554)

* remove dependence of Filter on Utilities (#710)

* move files

* docs for filter

* use amrex Pi instead of PeleC Pi

---------

Co-authored-by: Marc T. Henry de Frahan <[email protected]>
Co-authored-by: Jon Rood <[email protected]>
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