[Mono-dev] Proposed Patch - Google Native Client

[Zoltan Varga]

Hi,

  I think the runtime changes now look mostly ok, if nobody from the runtime
team has further
comments, I will commit them with some coding style etc. changes when I come
back from
holidays.

                                      Zoltan

On Fri, Jul 16, 2010 at 1:30 AM, Elijah Taylor <elijahtaylor at google.com>wrote:

> Hi, here's an updated patch with your feedback addressed.  I re-based the
> diff closer to head revision (r160382) to include the other changes of ours
> that already landed, as well as make sure we're still compatible with
> current Mono development.
>
> In general this diff should have a smaller impact on the .c files:
> mini-x86.c, exceptions-x86.c, tramp-x86.c specifically, and the Native
> Client changes are a little more grouped together rather than spread out.
>
> A couple of points separate from the feedback:
> 1) I fixed a bug in my implementation of genmdesc.pl changes, so that will
> be different from the previous patch
> 2) There's a small typo at head revision in mono/mini/tramp-x86.c which
> says "rethow" instead of "reth*r*ow" for your rethrow exception
> trampoline.  This is also fixed in my patch.
>
> As always feedback is appreciated from everyone.
>
>
> -Elijah
>
>
> On Tue, Jul 6, 2010 at 6:35 AM, Zoltan Varga <vargaz at gmail.com> wrote:
>
>> Hi,
>>
>>
>>> One possibility is to pad out all x86_prefix instructions to the nearest
>>> 32-byte boundary, but that could really bloat things depending on how often
>>> they're used.  Do you have any idea of the prefix to non-prefix instruction
>>> ratio?  It seems like it'd be pretty low based on looking at the code but I
>>> haven't looked at any actual metrics.
>>>
>>>
>> I think that would be ok, they are seldom used.
>>
>>                                  Zoltan
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.ximian.com/pipermail/mono-devel-list/attachments/20100723/b9f13545/attachment.html