Implement kat::apply and kat::invoke #86
Assignees
Labels
enhancement
New feature or request
Comments
|
In the mean time, commenting-out |
eyalroz
pushed a commit
that referenced
this issue
Nov 23, 2020
…ed using C++17): * Corrected `using` syntax for `conjunction`, `disjunction`, `bool_constant` and `negation` * Now using `kat::swap` in the swapping loop of `kat::array::swap` * Now using iterators in the swapping loop of `kat::array::swap` * Not trying to issue a trap instruction with C++17 in constexpr methods; with C++20 - issuing the instruction when guaranteed evaluation at run-time (and on the device) * Added: Missing template argument deducation guide for `kat::array` * Made sure `std::forward` and `std::move` are made accessible with `using` where necessary * Dropped a copy of an internal libstdc++ definition we were making by mistake * Avoiding warnings in `span.hpp` about a useless comparison of a 0-valued template argument (which only show up with C++17 enabled) * Removed inappropriately-placed `KAT_HD` markers within `if constexpr` statements * Properly marking `std::byte` with its namespace (rather than using just `byte`) * Properly marking `std::initializer_list` with its namespace (rather than using just `initializer_list`) * Added: A note about a missing implementation of kat::apply (see issue #86) * Avoiding a warning about a narrowing cast `int` -> `std::size_t` which the compiler should not be issuing (for i between 0 and 4) * Moved a type definition out of a doctest sub-testcase - another issue similar to what we had with `tuple_get` (this may already be a problem with C++14) * Some commented-out code and `#if FALSE`'es related to issue #87 (C++17 + swap troubles) * Some code formatting tweaks
eyalroz
pushed a commit
that referenced
this issue
May 28, 2021
…ng C++17 by default): * Corrected `using` syntax for `conjunction`, `disjunction`, `bool_constant` and `negation` * Now using `kat::swap` in the swapping loop of `kat::array::swap` * Now using iterators in the swapping loop of `kat::array::swap` * Not trying to issue a trap instruction with C++17 in constexpr methods; with C++20 - issuing the instruction when guaranteed evaluation at run-time (and on the device) * Added: Missing template argument deducation guide for `kat::array` * Made sure `std::forward` and `std::move` are made accessible with `using` where necessary * Dropped a copy of an internal libstdc++ definition we were making by mistake * Avoiding warnings in `span.hpp` about a useless comparison of a 0-valued template argument (which only show up with C++17 enabled) * Removed inappropriately-placed `KAT_HD` markers within `if constexpr` statements * Properly marking `std::byte` with its namespace (rather than using just `byte`) * Properly marking `std::initializer_list` with its namespace (rather than using just `initializer_list`) * Added: A note about a missing implementation of kat::apply (see issue #86) * Avoiding a warning about a narrowing cast `int` -> `std::size_t` which the compiler should not be issuing (for i between 0 and 4) * Moved a type definition out of a doctest sub-testcase - another issue similar to what we had with `tuple_get` (this may already be a problem with C++14) * Some commented-out code and `#if FALSE`'es related to issue #87 (C++17 + swap troubles) * Some code formatting tweaks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We currently have a non-functioning
kat::applyfor tuples - since there's nokat::invoke. Let's implement the latter to allow for the former.The text was updated successfully, but these errors were encountered: