The reason M5_DUMMY_RETURN didn't work is for gcc it's #defined to nothing
because in theory gcc respects __attribute__(no return), but apparently not in
this case.
Ali
On Mar 9, 2011, at 7:00 AM, Nilay wrote:
> IIRC, I was expecting some response from Ali as to why M5_DUMMY_RETURN
> should or should not work. I did not poke in any further. To me it is a
> compiler bug that we have to work with. I think return panic(""); works
> with both 4.2 and 4.4 series, but we probably do not want that.
>
> --
> Nilay
>
> On Tue, March 8, 2011 11:37 pm, nathan binkert wrote:
>> Nilay,
>>
>> I know that this is a way old e-mail, but did you ever figure this out?
>>
>> Nate
>>
>> On Fri, Dec 24, 2010 at 8:57 AM, Nilay Vaish <ni...@cs.wisc.edu> wrote:
>>> I tried M5_DUMMY_RETURN and it it not working. I checked its definition.
>>> It
>>> would evaluate to nothing, in which case I do not see why it should help
>>> in
>>> avoiding the warning. I tried putting a return statement before panic();
>>>
>>> return panic("not implemented");
>>>
>>> This works with GCC 4.2.2.
>>>
>>> I checked whether GCC has some recorded bug reports for this. I found
>>> the
>>> following two -
>>>
>>> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30988
>>> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42674
>>>
>>> I tried the provided codes. For the first one, 4.2.2 raises a warning
>>> but
>>> 4.4.0 does not. For the second one, 4.4.0 raises a warning but 4.2.2
>>> does
>>> not.
>>>
>>> --
>>> Nilay
>>>
>>> On Thu, 23 Dec 2010, Ali Saidi wrote:
>>>
>>>> A better solution would be to put M5_DUMMY_RETURN there. I know there'd
>>>> were some issues with some versions of gcc not respecting the attribute
>>>> no
>>>> return. This might be the case.
>>>>
>>>> Ali
>>>>
>>>> Sent from my ARM powered device
>>>>
>
> _______________________________________________
> m5-dev mailing list
> m5-dev@m5sim.org
> http://m5sim.org/mailman/listinfo/m5-dev
>
_______________________________________________
m5-dev mailing list
m5-dev@m5sim.org
http://m5sim.org/mailman/listinfo/m5-dev
