Author: [EMAIL PROTECTED]
Date: Fri Sep 26 08:53:42 2008
New Revision: 387

Modified:
    branches/bleeding_edge/SConstruct
    branches/bleeding_edge/src/objects.cc

Log:
Use -O9, tune for nocoma, assume at least a Pentium when generating code.
Review URL: http://codereview.chromium.org/4298

Modified: branches/bleeding_edge/SConstruct
==============================================================================
--- branches/bleeding_edge/SConstruct   (original)
+++ branches/bleeding_edge/SConstruct   Fri Sep 26 08:53:42 2008
@@ -53,7 +53,7 @@
        'CPPDEFINES':   ['ENABLE_DISASSEMBLER', 'DEBUG']
      },
      'mode:release': {
-      'CCFLAGS':      ['-O2']
+      'CCFLAGS':      ['-O9', '-mtune=nocona', '-march=pentium']
      },
      'wordsize:64': {
        'CCFLAGS':      ['-m32'],
@@ -188,7 +188,7 @@
        'LINKFLAGS':    ['-m32']
      },
      'mode:release': {
-      'CCFLAGS':      ['-O2']
+      'CCFLAGS':      ['-O2', '-mtune=nocona', '-march=pentium']
      },
      'mode:debug': {
        'CCFLAGS':      ['-g', '-O0']

Modified: branches/bleeding_edge/src/objects.cc
==============================================================================
--- branches/bleeding_edge/src/objects.cc       (original)
+++ branches/bleeding_edge/src/objects.cc       Fri Sep 26 08:53:42 2008
@@ -1691,7 +1691,7 @@
  PropertyAttributes JSObject::GetPropertyAttributeWithReceiver(
        JSObject* receiver,
        String* key) {
-  uint32_t index;
+  uint32_t index = 0;
    if (key->AsArrayIndex(&index)) {
      if (HasElementWithReceiver(receiver, index)) return NONE;
      return ABSENT;
@@ -1738,7 +1738,7 @@

  PropertyAttributes JSObject::GetLocalPropertyAttribute(String* name) {
    // Check whether the name is an array index.
-  uint32_t index;
+  uint32_t index = 0;
    if (name->AsArrayIndex(&index)) {
      if (HasLocalElement(index)) return NONE;
      return ABSENT;
@@ -2010,7 +2010,7 @@
    // ECMA-262, 3rd, 8.6.2.5
    ASSERT(name->IsString());

-  uint32_t index;
+  uint32_t index = 0;
    if (name->AsArrayIndex(&index)) {
      return DeleteElement(index);
    } else {

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to