Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(816)

Side by Side Diff: src/regexp-macro-assembler-tracer.cc

Issue 10386090: Implement loop for global regexps in regexp assembler. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: fix bugs, add tests, port to x64 and arm. Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2008 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
11 // with the distribution. 11 // with the distribution.
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
96 } 96 }
97 97
98 98
99 void RegExpMacroAssemblerTracer::PushBacktrack(Label* label) { 99 void RegExpMacroAssemblerTracer::PushBacktrack(Label* label) {
100 PrintF(" PushBacktrack(label[%08x]);\n", LabelToInt(label)); 100 PrintF(" PushBacktrack(label[%08x]);\n", LabelToInt(label));
101 assembler_->PushBacktrack(label); 101 assembler_->PushBacktrack(label);
102 } 102 }
103 103
104 104
105 void RegExpMacroAssemblerTracer::Succeed() { 105 void RegExpMacroAssemblerTracer::Succeed() {
106 PrintF(" Succeed();\n"); 106 PrintF(" Succeed();%s\n",
107 assembler_->global() ? " [restart for global match]" : "");
Erik Corry 2012/05/22 08:32:46 It's a bit ugly that this will print "restart..."
107 assembler_->Succeed(); 108 assembler_->Succeed();
108 } 109 }
109 110
110 111
111 void RegExpMacroAssemblerTracer::Fail() { 112 void RegExpMacroAssemblerTracer::Fail() {
112 PrintF(" Fail();\n"); 113 PrintF(" Fail();%s\n",
114 assembler_->global() ? " [return success count]" : "");
113 assembler_->Fail(); 115 assembler_->Fail();
114 } 116 }
115 117
116 118
117 void RegExpMacroAssemblerTracer::PopRegister(int register_index) { 119 void RegExpMacroAssemblerTracer::PopRegister(int register_index) {
118 PrintF(" PopRegister(register=%d);\n", register_index); 120 PrintF(" PopRegister(register=%d);\n", register_index);
119 assembler_->PopRegister(register_index); 121 assembler_->PopRegister(register_index);
120 } 122 }
121 123
122 124
(...skipping 326 matching lines...) Expand 10 before | Expand all | Expand 10 after
449 return assembler_->Implementation(); 451 return assembler_->Implementation();
450 } 452 }
451 453
452 454
453 Handle<HeapObject> RegExpMacroAssemblerTracer::GetCode(Handle<String> source) { 455 Handle<HeapObject> RegExpMacroAssemblerTracer::GetCode(Handle<String> source) {
454 PrintF(" GetCode(%s);\n", *(source->ToCString())); 456 PrintF(" GetCode(%s);\n", *(source->ToCString()));
455 return assembler_->GetCode(source); 457 return assembler_->GetCode(source);
456 } 458 }
457 459
458 }} // namespace v8::internal 460 }} // namespace v8::internal
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698