More constexpr!
This MR focuses in FieldVector
, FieldMatrix
and iterator utilities. In particular, this is a step towards a constexpr
reference elements in dune-geometry.
Note that this needs !1388 (closed) to be resolved.
Merge request reports
Activity
added feature label
requested review from @simon.praetorius
assigned to @santiago.ospina
- Resolved by Christoph Grüninger
Have a look into !1401 . Let's do this transition to more constexpr step by step, I would suggest.
- Resolved by Simon Praetorius
- Resolved by Simon Praetorius
- Resolved by Santiago Ospina De Los Ríos
- Resolved by Christoph Grüninger
I am not convinced that re-implementing basic math stuff is the right approach. Rely on C++23 or even C++26 if you really need
constexpr
for exceptions and other difficult cases. With other words, guard theconstexpr
that do not compile with C++20 with feature macros or language macros.
added 1 commit
- 3996dc6c - Throw RangeError exception in bounds check instead of using DUNE_THROW
- Resolved by Santiago Ospina De Los Ríos
- Resolved by Santiago Ospina De Los Ríos
This MR lacks a change log entry.
added 69 commits
-
3996dc6c...fa579ea1 - 66 commits from branch
master
- 1a16e388 - Add more constexpr qualifiers
- a15dbfd2 - More constexpr: backport constexpr versions of std::abs and std::sqrt
- cf1b6cb4 - Add integral and floating point constraints
Toggle commit list-
3996dc6c...fa579ea1 - 66 commits from branch
added 2 commits
- Resolved by Santiago Ospina De Los Ríos
mentioned in merge request !1485 (merged)
- Resolved by Santiago Ospina De Los Ríos
What is needed to make progress for this topic?
added 41 commits
-
0b36e315...8b31a737 - 34 commits from branch
master
- 39bca8f4 - Add more constexpr qualifiers
- ad350c7b - More constexpr: backport constexpr versions of std::abs and std::sqrt
- a8e061dc - Add integral and floating point constraints
- 4f282d2a - Move back-port within macro
- f04e57fb - Format code
- a641a08b - Use ADL to resolve std abs
- ce8abde6 - Remove const-eval preprocessor check
Toggle commit list-
0b36e315...8b31a737 - 34 commits from branch
added 1 commit
- 96212ee5 - Remove superfluous endif prerpocessor directive
requested review from @gruenich and removed review request for @simon.praetorius
enabled an automatic merge when all merge checks for f6843fb9 pass
added 15 commits
-
f6843fb9...8b31a737 - 14 commits from branch
master
- 00256cb8 - Add more constexpr
-
f6843fb9...8b31a737 - 14 commits from branch
enabled an automatic merge when all merge checks for 00256cb8 pass
mentioned in commit 8f1177e4