Skip to content

Commit

Permalink
Documentation: Fix malformed table (#4305)
Browse files Browse the repository at this point in the history
Also fix an incorrect assertion due to code changes. Since we no longer
support C++ < 17, we simply remove the assertion.
  • Loading branch information
WeiqunZhang authored Jan 22, 2025
1 parent d4d9214 commit bd2d730
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 11 deletions.
4 changes: 2 additions & 2 deletions Docs/sphinx_documentation/source/BuildingAMReX.rst
Original file line number Diff line number Diff line change
Expand Up @@ -464,10 +464,10 @@ The list of available options is reported in the :ref:`table <tab:cmakevar>` bel
| AMReX_LINEAR_SOLVERS | Build AMReX linear solvers | YES | YES, NO |
+------------------------------+-------------------------------------------------+-------------------------+-----------------------+
| AMReX_LINEAR_SOLVERS_INCFLO | Build AMReX linear solvers for incompressible | YES | YES, NO |
| | flow | |
| | flow | | |
+------------------------------+-------------------------------------------------+-------------------------+-----------------------+
| AMReX_LINEAR_SOLVERS_EM | Build AMReX linear solvers for electromagnetic | YES | YES, NO |
| | solvers | |
| | solvers | | |
+------------------------------+-------------------------------------------------+-------------------------+-----------------------+
| AMReX_AMRDATA | Build data services | NO | YES, NO |
+------------------------------+-------------------------------------------------+-------------------------+-----------------------+
Expand Down
9 changes: 0 additions & 9 deletions Src/Base/AMReX_CTOParallelForImpl.H
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,6 @@ struct CompileTimeOptions {
using list_type = TypeList<std::integral_constant<int, ctr>...>;
};

#if (__cplusplus >= 201703L)

namespace detail
{
template<class F, int... ctr>
Expand Down Expand Up @@ -76,8 +74,6 @@ namespace detail
}
}

#endif

/**
* \brief Compile time optimization of kernels with run time options.
*
Expand Down Expand Up @@ -184,15 +180,10 @@ void AnyCTO ([[maybe_unused]] TypeList<CTOs...> list_of_compile_time_options,
std::array<int,sizeof...(CTOs)> const& runtime_options,
L&& l, Fs&&...cto_functs)
{
#if (__cplusplus >= 201703L)
detail::AnyCTO_helper1(std::forward<L>(l),
CartesianProduct(typename CTOs::list_type{}...),
runtime_options,
std::forward<Fs>(cto_functs)...);
#else
amrex::ignore_unused(runtime_options, l, f);
static_assert(std::is_integral<F>::value, "This requires C++17");
#endif
}

template <int MT, typename T, class F, typename... CTOs>
Expand Down

0 comments on commit bd2d730

Please sign in to comment.