On Dec 11, 2023, Jeff Law <jeffreya...@gmail.com> wrote: >> >> for gcc/ChangeLog >> PR target/112334 >> * builtins.h (target_builtins): Add fields for apply_args_size >> and apply_result_size. >> * builtins.cc (apply_args_size, apply_result_size): Cache >> results in fields rather than in static variables. >> (get_apply_args_size, set_apply_args_size): New. >> (get_apply_result_size, set_apply_result_size): New. > OK.
Thanks, I put this bit in. >> untyped calls: use wrapper class type for implicit plus_one >> Instead of get and set macros to apply a delta, use a single macro >> that resorts to a temporary wrapper class to apply it. >> To be combined (or not) with the previous patch. > I'd be OK with this as well. That conditional made me doubt that this was meant as approval, so I did *not* put this one in ;-) If there's firmer/broader buy-in, I'd be glad to put it in as well. -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer More tolerance and less prejudice are key for inclusion and diversity Excluding neuro-others for not behaving ""normal"" is *not* inclusive
