Changeset 160091 in webkit

Timestamp:

Dec 4, 2013 6:42:09 AM (10 years ago)

Author:

commit-queue@webkit.org

Message:

Avoid to add zero offset in BaseIndex.
​https://bugs.webkit.org/show_bug.cgi?id=125215

Patch by László Langó <​lango@inf.u-szeged.hu> on 2013-12-04
Reviewed by Michael Saboff.

When using cloop do not generate offsets additions for BaseIndex if the offset is zero.

• offlineasm/cloop.rb:

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• offlineasm/cloop.rb (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2013-12-04  László Langó  <lango@inf.u-szeged.hu>
+
+        Avoid to add zero offset in BaseIndex.
+        https://bugs.webkit.org/show_bug.cgi?id=125215
+
+        Reviewed by Michael Saboff.
+
+        When using cloop do not generate offsets additions for BaseIndex if the offset is zero.
+
+        * offlineasm/cloop.rb:
+
 2013-12-04  Peter Molnar  <pmolnar.u-szeged@partner.samsung.com>
 

trunk/Source/JavaScriptCore/offlineasm/cloop.rb

@@ -262 +262 @@
     end
     def pointerExpr
-        "#{base.clValue(:int8Ptr)} + (#{index.clValue} << #{scaleShift}) + #{offset.clValue}"
+        if offset.value == 0
+            "#{base.clValue(:int8Ptr)} + (#{index.clValue} << #{scaleShift})"
+        else
+            "#{base.clValue(:int8Ptr)} + (#{index.clValue} << #{scaleShift}) + #{offset.clValue}"
+        end
     end
     def int8MemRef