On 10/20/22 02:27, Luis Machado wrote:
> Hi,
> 
> On 10/19/22 19:54, Tom de Vries wrote:
>> On 10/12/22 16:53, Tom de Vries wrote:
>>> diff --git a/gdb/testsuite/gdb.arch/arm-disp-step.S 
>>> b/gdb/testsuite/gdb.arch/arm-disp-step.S
>>> index b67e83e56e0..3cfb6063f40 100644
>>> --- a/gdb/testsuite/gdb.arch/arm-disp-step.S
>>> +++ b/gdb/testsuite/gdb.arch/arm-disp-step.S
>>> @@ -391,3 +391,4 @@ test_add_rn_pc_end:
>>>       bx lr
>>>       .size test_add_rn_pc, .-test_add_rn_pc
>>>   #endif
>>> +    .section    .note.GNU-stack,"",@progbits
>>
>> Hi,
>>
>> I realize now that this'll probably need to be:
>> ...
>> +    .section    .note.GNU-stack,"",%progbits
>> ...
>> and the same for the dwarf assembly test-cases when using target arm*
>>
>> See f.i. 9a5911c08be ("gdb/testsuite/gdb.dwarf2: Replace @ with % for 
>> ARM compatability.")
>>
>> What I cannot figure out if that holds as well for aarch64 archs: does 
>> that one also use %progbits instead of @progbits?
>>
>> Luis, do you happen to know?
> 
>  From what I see on aarch64, it uses @progbits.
> 
> .section        .debug_info,"",@progbits

Ack, thanks for checking.

Committed as attached.

Thanks,
- Tom