On Thu, Aug 11, 2011 at 10:41 PM, Hakan Ardo <ha...@debian.org> wrote:
> On Wed, Aug 10, 2011 at 6:42 PM, Hakan Ardo <ha...@debian.org> wrote: > > > > I'm not sure why test_instance works on trunk actually, but the way to > > get it to work on this branch I think is to make the heap optimizer > > aware of the setfield_gc's produced by forcing p22 and then have the > > heap optimizer remove the getfield_gc_pure. Using the pure operations > > optimizer here is not straight forwars as there is no > > getfield_gc_pure(p22). > > This issues does not show up on trunk because of is_forced_virtual() > keeps track of forced virtuals and treats them separately. I dont > think we want specialize loops to inputargs beeing forced virtuals and > guard for this condition at the end of bridges... > > -- > Håkan Ardö > The only motivation for `is_forced_virtual` was to have fields that are pure be kept as known values (to prevent re-reading), since when you force a virtual the setfields don't go through the heap cache (which they probably should, TBH, since we know exactly what fields a call can force). Alex -- "I disapprove of what you say, but I will defend to the death your right to say it." -- Evelyn Beatrice Hall (summarizing Voltaire) "The people's good is the highest law." -- Cicero
_______________________________________________ pypy-dev mailing list pypy-dev@python.org http://mail.python.org/mailman/listinfo/pypy-dev
