Changeset 129434 in webkit for trunk/Source/JavaScriptCore/offlineasm/parser.rb

Timestamp:

Sep 24, 2012, 5:38:51 PM (13 years ago)

Author:

[email protected]

Message:

Add cloopDo instruction for debugging the llint C++ backend.
​https://bugs.webkit.org/show_bug.cgi?id=97502.

Reviewed by Geoffrey Garen.

• offlineasm/cloop.rb:
• offlineasm/instructions.rb:
• offlineasm/parser.rb:

File:

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

trunk/Source/JavaScriptCore/offlineasm/parser.rb

@@ -104 +104 @@
         when /\A\n/
             # We've found a '\n'.  Emit the last comment recorded if appropriate:
-            if $enableInstrAnnotations and annotation
+            # We need to parse annotations regardless of whether the backend does
+            # anything with them or not. This is because the C++ backend may make
+            # use of this for its cloopDo debugging utility even if
+            # enableInstrAnnotations is not enabled.
+            if annotation
                 result << Annotation.new(CodeOrigin.new(fileName, lineNumber),
                                          annotationType, annotation)