[gdb/testsuite] Fix calls in gdb.arch/thumb2-it.exp
Commit Message
Hi,
Tests in gdb.arch/thumb2-it.exp call functions defined in assembly
without type debugging information. Since
7022349d5c86bae74b49225515f42d2e221bd368 this triggers an error which
leads to many tests to FAIL. This patch cast the call to indicate the
return type of the functions when calling them.
ChangeLog entry is as follows:
*** gdb/testsuite/ChangeLog ***
2017-09-06 Thomas Preud'homme <thomas.preudhomme@arm.com>
* gdb.arch/thumb2-it.exp: Cast call to assembly defined function.
All tests in that exp file PASS after that change.
Is this ok for master?
Best regards,
Thomas
Comments
On 09/06/2017 04:26 PM, Thomas Preudhomme wrote:
> Hi,
>
> Tests in gdb.arch/thumb2-it.exp call functions defined in assembly
> without type debugging information. Since
> 7022349d5c86bae74b49225515f42d2e221bd368
I find Linux's style to refer to past commits helpful when
reading commit logs. I.e., also mention the commit subject, like:
... Since 7022349d5c86 ("Stop assuming no-debug-info functions
return int"), this triggers ...
This gives a clue to humans what the commit is about.
> this triggers an error which
> leads to many tests to FAIL. This patch cast the call to indicate the
> return type of the functions when calling them.
>
> Is this ok for master?
OK.
Thanks,
Pedro Alves
On 06/09/17 16:31, Pedro Alves wrote:
> On 09/06/2017 04:26 PM, Thomas Preudhomme wrote:
>> Hi,
>>
>> Tests in gdb.arch/thumb2-it.exp call functions defined in assembly
>> without type debugging information. Since
>> 7022349d5c86bae74b49225515f42d2e221bd368
>
> I find Linux's style to refer to past commits helpful when
> reading commit logs. I.e., also mention the commit subject, like:
>
> ... Since 7022349d5c86 ("Stop assuming no-debug-info functions
> return int"), this triggers ...
>
> This gives a clue to humans what the commit is about.
Indeed, thanks for the suggestion.
>
>> this triggers an error which
>> leads to many tests to FAIL. This patch cast the call to indicate the
>> return type of the functions when calling them.
>
>>
>> Is this ok for master?
>
> OK.
Pushed.
Best regards,
Thomas
@@ -58,7 +58,7 @@ proc test_it_block { func } {
return
}
- gdb_test "call ${func}()" "Breakpoint.*@ Setup.*" "$func, call"
+ gdb_test "call (int) ${func}()" "Breakpoint.*@ Setup.*" "$func, call"
set expected 0
set reached 0
@@ -155,7 +155,7 @@ for { set i 1 } { $i <= 8 } { incr i } {
}
gdb_breakpoint "*it_breakpoints"
-gdb_test "call it_breakpoints()" "Breakpoint.*"
+gdb_test "call (int) it_breakpoints()" "Breakpoint.*"
for { set i 1 } { $i <= 7 } { incr i } {
test_it_break ${i}
}