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

Unified Diff: src/jsregexp.cc

Issue 21014: RegExp: Fix missing adaption of debug check after changing definition of a Trace being trivial. (Closed)
Patch Set: Created 11 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/jsregexp.cc
diff --git a/src/jsregexp.cc b/src/jsregexp.cc
index c737b027195cbb96c2f4bb0a0f3234ad5a6fda90..c827063ef7d790cc1240f8b3de2b31b2e143f62f 100644
--- a/src/jsregexp.cc
+++ b/src/jsregexp.cc
@@ -1507,12 +1507,7 @@ void Trace::PerformDeferredActions(RegExpMacroAssembler* assembler,
void Trace::Flush(RegExpCompiler* compiler, RegExpNode* successor) {
RegExpMacroAssembler* assembler = compiler->macro_assembler();
- ASSERT(actions_ != NULL ||
- cp_offset_ != 0 ||
- backtrack() != NULL ||
- characters_preloaded_ != 0 ||
- quick_check_performed_.characters() != 0 ||
- bound_checked_up_to_ != 0);
+ ASSERT(!is_trivial());
if (actions_ == NULL && backtrack() == NULL) {
// Here we just have some deferred cp advances to fix and we are back to
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698