Re: [PATCH] untyped calls: enable target switching [PR112334]

2023-12-20 Thread Alexandre Oliva
On Dec 20, 2023, Jeff Law wrote: > Sorry it was meant to be an ACK for the trunk for both patches. Aah, I see. So this is what I installed last night, upon seeing your message. untyped calls: use wrapper class type for implicit plus_one Instead of get and set macros to apply a delta, use a s

Re: [PATCH] untyped calls: enable target switching [PR112334]

2023-12-19 Thread Jeff Law
On 12/11/23 21:44, Alexandre Oliva wrote: 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

Re: [PATCH] untyped calls: enable target switching [PR112334]

2023-12-11 Thread Alexandre Oliva
On Dec 11, 2023, Jeff Law 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_app

Re: [PATCH] untyped calls: enable target switching [PR112334]

2023-12-11 Thread Jeff Law
On 12/1/23 08:10, Alexandre Oliva wrote: On Dec 1, 2023, Alexandre Oliva wrote: Also tested on arm-eabi, but it's *not* enough (or needed) to fix the PR, there's another bug lurking there, with a separate patch coming up. Here it is. The computation of apply_args_size and apply_re

[PATCH] untyped calls: enable target switching [PR112334]

2023-12-01 Thread Alexandre Oliva
On Dec 1, 2023, Alexandre Oliva wrote: > Also tested on arm-eabi, but it's *not* enough (or needed) to fix the > PR, there's another bug lurking there, with a separate patch coming > up. Here it is. The computation of apply_args_size and apply_result_size is saved in a static variable, s