<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>18643</bug_id>
          
          <creation_ts>2008-04-20 16:36:14 -0700</creation_ts>
          <short_desc>SQUIRRELFISH: need to support implicit function calls (valueOf, toString)</short_desc>
          <delta_ts>2008-04-29 22:15:40 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>JavaScriptCore</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>18624</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Maciej Stachowiak">mjs</reporter>
          <assigned_to name="Oliver Hunt">oliver</assigned_to>
          <cc>oliver</cc>
    
    <cc>zwarich</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>78342</commentid>
    <comment_count>0</comment_count>
    <who name="Maciej Stachowiak">mjs</who>
    <bug_when>2008-04-20 16:36:14 -0700</bug_when>
    <thetext>SquirrelFish does not yet properly handle property getters and setters.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>78692</commentid>
    <comment_count>1</comment_count>
    <who name="Oliver Hunt">oliver</who>
    <bug_when>2008-04-23 13:14:03 -0700</bug_when>
    <thetext>The problem for these implicit function calls (valueOf is called by toNumber, toString by +, etc, getters/setters by resolve) is that they may require a function call in the VM, which may inturn result in resizing of the RegisterFile which would invalidate state in the originating privateExecute frame.  Adding the required guards around all call sites would almost certainly be a perf hit but maybe possible at a few places (resolve?)

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>78731</commentid>
    <comment_count>2</comment_count>
    <who name="Cameron Zwarich (cpst)">zwarich</who>
    <bug_when>2008-04-23 21:20:47 -0700</bug_when>
    <thetext>This is causing a lot of failures in the remaining JavaScriptCore test failures, e.g. the instanceof tests. We should really find a way to fix this without too large of a performance regression.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>78748</commentid>
    <comment_count>3</comment_count>
    <who name="Maciej Stachowiak">mjs</who>
    <bug_when>2008-04-24 00:50:33 -0700</bug_when>
    <thetext>I&apos;m not sure valueOf / toString are quire the same issue as getters and setters, and we might not want to solve them the exact same way necessarily.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>79262</commentid>
    <comment_count>4</comment_count>
    <who name="Oliver Hunt">oliver</who>
    <bug_when>2008-04-29 22:15:40 -0700</bug_when>
    <thetext>Committed r32669 and r32718</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>