[RFA] Change gdbarch_return_value to take function type instead of return value type
Andrew Cagney
cagney@gnu.org
Mon Oct 25 23:18:00 GMT 2004
Corinna Vinschen wrote:
> On Oct 13 12:15, Corinna Vinschen wrote:
>
>>No, I won't. I found that my patch missed the calls of using_struct_return
>>in eval.c. These calls only happen in case of a OP_OBJC_MSGCALL operation.
>>
>>At line 841 in eval.c, in function evaluate_subexp_standard() I found this:
>>
>> /* If we found a method with symbol information, check to see
>> if it returns a struct. Otherwise assume it doesn't. */
>>
>> if (method)
>> {
>> [...]
>> struct_return = using_struct_return (value_type, using_gcc);
>> }
>> else if (expect_type != NULL)
>> {
>> struct_return = using_struct_return (check_typedef (expect_type), using_gcc);
>> }
>>
>>While the `if (method)' branch is simple, the `else if' branch can't be
>>solved easily. evaluate_subexp_standard just gets the expected return type
>>as parameter.
>>
>>Does anybody know how to get around this problem? For now I can only see
>>one way, redefine gdbarch_return_value to get both, function type and return
>>value type.
>
>
> Ok, since I didn't get a reply to this for a week, I take it that nobody
> has a better idea. I've changed the patch according to my above suggestion.
I believe JimB discussed this with you off list, I might as well summarize.
For the above, just locally construct a function-type from the
value_type / expect_type and then pass that in (ref make_function_type).
That both avoids the need for the extra parameter, and keeps things
simple and consistent.
Andrew
More information about the Gdb-patches
mailing list