• d-bugmail puremagic.com (23/23) Jun 04 2011 http://d.puremagic.com/issues/show_bug.cgi?id=6103
• d-bugmail puremagic.com (13/13) Dec 28 2011 http://d.puremagic.com/issues/show_bug.cgi?id=6103

d-bugmail puremagic.com writes:

http://d.puremagic.com/issues/show_bug.cgi?id=6103

           Summary: hasFinalizer field in ClassInfo
           Product: D
           Version: D2
          Platform: Other
        OS/Version: Windows
            Status: NEW
          Keywords: performance
          Severity: enhancement
          Priority: P2
         Component: DMD
        AssignedTo: nobody puremagic.com
        ReportedBy: dsimcha yahoo.com



ClassInfo needs a hasFinalizer field that is true iff the class or its ancestor
has a non-empty finalizer.  This way, instead of conservatively flagging all
GC-allocated objects with GC.BlkAttr.FINALIZE, only those that need to be
finalized could be flagged.  On this benchmark
(https://github.com/dsimcha/druntime/blob/master/gcBench/tree1.d) avoiding
calling empty finalizers results in about a 20% speedup.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

d-bugmail puremagic.com writes:

http://d.puremagic.com/issues/show_bug.cgi?id=6103


David Simcha <dsimcha yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |INVALID



I'm marking this invalid even though it could be implemented because I realized
it won't be useful.  rt_finalize needs to be called to finalize the monitor
object if one exists, even if the class has no explicit finalizer.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------