On 5/19/23 08:02, Bernhard Reutner-Fischer via Gcc-patches wrote:
On 19 May 2023 07:58:48 CEST, "SenthilKumar.Selvaraj--- via Gcc-patches" <gcc-patches@gcc.gnu.org> wrote: Just a nit:+static bool +avr_addr_space_zero_address_valid (addr_space_t as ATTRIBUTE_UNUSED) +{ + return flag_delete_null_pointer_checks == 0; +}Since we are c++ nowadays, you can omit the parameter name for unused arguments. I.e.: static bool avr_addr_space_zero_address_valid (addr_space_t) {
Right. And I strongly prefer that over ATTRIBUTE_UNUSED. So OK for the trunk with that change. jeff
