Changeset 200666 in webkit for trunk/Source/JavaScriptCore/offlineasm/asm.rb

Timestamp:

May 10, 2016, 6:56:00 PM (9 years ago)

Author:

[email protected]

Message:

REGRESSION(r200447): Unable to build C_LOOP with clang version 800.0.12 or higher
​https://bugs.webkit.org/show_bug.cgi?id=157549

Reviewed by Keith Miller.

Disable debug annotations for C_LOOP builds. They are inline assembly directives,
unnecessary and they cause syntax errors.

• offlineasm/asm.rb:

File:

• trunk/Source/JavaScriptCore/offlineasm/asm.rb (modified) (1 diff)

trunk/Source/JavaScriptCore/offlineasm/asm.rb

@@ -348 +348 @@
         forSettings(computeSettingsCombinations(ast)[configIndex], ast) {
             | concreteSettings, lowLevelAST, backend |
+
+            # There could be multiple backends we are generating for, but the C_LOOP is
+            # always by itself so this check to turn off $enableDebugAnnotations won't
+            # affect the generation for any other backend.
+            if backend == "C_LOOP"
+                $enableDebugAnnotations = false
+            end
+
             lowLevelAST = lowLevelAST.resolve(*buildOffsetsMap(lowLevelAST, offsetsList))
             lowLevelAST.validate