[Mono-dev] mono inlining

[crashfourit]

Rodrigo Kumpera wrote:
> 
> The _MonoMethod struct is instantiated too many times on during runtime to
> have information
> added to it. The extra space would cause a huge overhead.
> 
> The other issue is that the way it's doing is resolving too much metadata
> information. It
> should be done as lazily as possible.
> 
> I have a similar need that I'm coding right now, once I add the required
> code to do so, your patch will
> get a lot simpler.
> 
> 
> 
> On Mon, Jan 19, 2009 at 5:47 PM, crashfourit <crashfourit at gmail.com>
> wrote:
> 
>>
>>
>>
>> Rodrigo Kumpera wrote:
>> >
>> > This is the wrong approach to check for the assembly, you should first
>> > strcmp
>> > for it and only after that cache the result. Don't load the assembly if
>> > it's
>> > possible to avoid so.
>> >
>> > On Mon, Jan 19, 2009 at 8:04 AM, crashfourit <crashfourit at gmail.com>
>> > wrote:
>> >
>> >>
>> >> Here is my updated patch.
>> >>
>> >> http://www.nabble.com/file/p21539945/trunk_mono.diff trunk_mono.diff
>> >>
>> >> What do you think?
>> >> --
>> >> View this message in context:
>> >> http://www.nabble.com/mono-inlining-tp21510132p21539945.html
>> >> Sent from the Mono - Dev mailing list archive at Nabble.com.
>> >>
>> >> _______________________________________________
>> >> Mono-devel-list mailing list
>> >> Mono-devel-list at lists.ximian.com
>> >> http://lists.ximian.com/mailman/listinfo/mono-devel-list
>> >>
>> >
>> > _______________________________________________
>> > Mono-devel-list mailing list
>> > Mono-devel-list at lists.ximian.com
>> > http://lists.ximian.com/mailman/listinfo/mono-devel-list
>> >
>> >
>> Here is another updated patch.
>> Hope it is to your liking.
>> http://www.nabble.com/file/p21550206/trunk_mono.diff trunk_mono.diff
>> --
>> View this message in context:
>> http://www.nabble.com/mono-inlining-tp21510132p21550206.html
>> Sent from the Mono - Dev mailing list archive at Nabble.com.
>>
>> _______________________________________________
>> Mono-devel-list mailing list
>> Mono-devel-list at lists.ximian.com
>> http://lists.ximian.com/mailman/listinfo/mono-devel-list
>>
> 
> _______________________________________________
> Mono-devel-list mailing list
> Mono-devel-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-devel-list
> 
> 
I'm guessing you like my patch. :) Well, the general concept of it. I'd love
to see the code you are adding that will make my patch simpler.

Thing objective is that I'd love to figure out how to do struct
decomposition so that methods with structs as formal parameters will be able
to be inlined. 
-- 
View this message in context: http://www.nabble.com/mono-inlining-tp21510132p21550920.html
Sent from the Mono - Dev mailing list archive at Nabble.com.