Title: [126886] trunk/Source/_javascript_Core
- Revision
- 126886
- Author
- [email protected]
- Date
- 2012-08-28 09:02:03 -0700 (Tue, 28 Aug 2012)
Log Message
[GTK] LLint build fails with -g -02
https://bugs.webkit.org/show_bug.cgi?id=90098
Patch by Alban Browaeys <[email protected]> on 2012-08-28
Reviewed by Filip Pizlo.
Avoid duplicate offsets for llint, discarding them.
* offlineasm/offsets.rb:
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (126885 => 126886)
--- trunk/Source/_javascript_Core/ChangeLog 2012-08-28 15:58:21 UTC (rev 126885)
+++ trunk/Source/_javascript_Core/ChangeLog 2012-08-28 16:02:03 UTC (rev 126886)
@@ -1,3 +1,14 @@
+2012-08-28 Alban Browaeys <[email protected]>
+
+ [GTK] LLint build fails with -g -02
+ https://bugs.webkit.org/show_bug.cgi?id=90098
+
+ Reviewed by Filip Pizlo.
+
+ Avoid duplicate offsets for llint, discarding them.
+
+ * offlineasm/offsets.rb:
+
2012-08-27 Sheriff Bot <[email protected]>
Unreviewed, rolling out r126836.
Modified: trunk/Source/_javascript_Core/offlineasm/offsets.rb (126885 => 126886)
--- trunk/Source/_javascript_Core/offlineasm/offsets.rb 2012-08-28 15:58:21 UTC (rev 126885)
+++ trunk/Source/_javascript_Core/offlineasm/offsets.rb 2012-08-28 16:02:03 UTC (rev 126886)
@@ -61,7 +61,7 @@
#
# offsetsAndConfigurationIndex(ast, file) ->
-# [[offsets, index], ...]
+# {[offsets, index], ...}
#
# Parses the offsets from a file and returns a list of offsets and the
# index of the configuration that is valid in this build target.
@@ -69,7 +69,7 @@
def offsetsAndConfigurationIndex(file)
endiannessMarkerBytes = nil
- result = []
+ result = {}
def readInt(endianness, bytes)
if endianness == :little
@@ -155,13 +155,14 @@
| data |
offsets << readInt(endianness, data)
}
- result << [offsets, index]
+ if not result.has_key?(offsets)
+ result[offsets] = index
+ end
}
end
}
raise MissingMagicValuesException unless result.length >= 1
- raise if result.map{|v| v[1]}.uniq.size < result.map{|v| v[1]}.size
result
end
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
