On 03/05/2026 12:43, Jeffrey Law wrote:
On 2/10/2026 9:51 AM, Nathan Sidwell wrote:
The riscv builtins are defined via DIRECT_BUILTIN and
DIRECT_NO_TARGET_BUILTIN, which distinguish as to whether the builtin returns
a result or not. But that can be directly determined from the builtin's
function type (whether it returns void or not).
This patch removes the NO_TARGET macros and adjusts all the .def files
appropriately. The riscv_builtin_description structure is adjusted to have a
bool indicating it has no target, and this is computed during builtin
initialization. The fields are reordered for better packing on a 64-bit host.
OK after a re-test on on a modern trunk head.
jeff
I have pushed the patch, tested on one of the compute farm machines
nathan
--
Nathan Sidwell