Re: [Qemu-devel] [PATCH] target-moxie: Fix pointer-to-integer conversion (MinGW-w64)

[Blue Swirl]

On Thu, Mar 28, 2013 at 7:00 PM, Anthony Green <address@hidden> wrote:
> Hi Stefan,
>
> This change is fine by me.
>
> It's not clear to me, as the author/maintainer of the moxie port, what
> my responsibility/authority is.  Do I simply reply to patches like
> this with...
>
> Looks good to me.
> Signed-off-by: Anthony Green <address@hidden>

I think Acked-by: or Reviewed-by: tags are fine too.

>
> And then somebody commits it to the tree?

Yes, like I'm going to do now. You could also queue the patches to
your tree (for example, if they collide with your work in progress) by
using git am and then send the patches as part of your patch set or
pull request. In that case, the From: field indicating the original
submitter should be kept intact, but git am should do that for you.

In some cases, like trivial patches, fixing a build breakage or
general refactoring which only needs minor changes to several targets,
the committers do not wait too long for OK from target maintainers.
Positive reviews with tags of course increase confidence to the patch
and a Nack is always possible.

>
> Thanks!
>
> AG
>
>
> On Sun, Mar 24, 2013 at 4:04 AM, Stefan Weil <address@hidden> wrote:
>> The type cast must use tcg_target_long instead of long.
>> This makes a difference for hosts where sizeof(long) != sizeof(void *).
>>
>> Cc: Anthony Green <address@hidden>
>> Cc: Blue Swirl <address@hidden>
>> Signed-off-by: Stefan Weil <address@hidden>
>> ---
>>  target-moxie/translate.c |    2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/target-moxie/translate.c b/target-moxie/translate.c
>> index 34f166e..cc02bd3 100644
>> --- a/target-moxie/translate.c
>> +++ b/target-moxie/translate.c
>> @@ -133,7 +133,7 @@ static inline void gen_goto_tb(CPUMoxieState *env, 
>> DisasContext *ctx,
>>          !ctx->singlestep_enabled) {
>>          tcg_gen_goto_tb(n);
>>          tcg_gen_movi_i32(cpu_pc, dest);
>> -        tcg_gen_exit_tb((long) tb + n);
>> +        tcg_gen_exit_tb((tcg_target_long)tb + n);
>>      } else {
>>          tcg_gen_movi_i32(cpu_pc, dest);
>>          if (ctx->singlestep_enabled) {
>> --
>> 1.7.10.4
>>