Experimental: fix a bug in our handling of the short-circuit boolean...

src/codegen-ia32.h

 // Copyright 2006-2008 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 173 matching lines @@
     JumpTarget* target = where ? true_target_ : false_target_;
     if (target->is_bound()) {
       target->Jump();
     } else {
       target->Bind();
     }
     is_used_ = true;
     true_is_fall_through_ = where;
   }
 
+  // Mark this jump target as used as if Goto had been called, but
+  // without generating a jump or binding a label (the control effect
+  // should have already happened).  This is used when the left
+  // subexpression of the short-circuit boolean operators are
+  // compiled.
+  void Use(bool where) {
+    ASSERT(!is_used_);
+    ASSERT((where ? true_target_ : false_target_)->is_bound());
+    is_used_ = true;
+    true_is_fall_through_ = where;
+  }
+
   // Swap the true and false targets but keep the same actual label as
   // the fall through.  This is used when compiling negated
   // expressions, where we want to swap the targets but preserve the
   // state.
   void Invert() {
     JumpTarget* temp_target = true_target_;
     true_target_ = false_target_;
     false_target_ = temp_target;
 
     true_is_fall_through_ = !true_is_fall_through_;
@@ skipping 433 matching lines @@
   cgen_->set_in_spilled_code(false);
   GetValue(typeof_state);
   cgen_->frame()->SpillAll();
   cgen_->set_in_spilled_code(true);
 }
 
 
 } }  // namespace v8::internal
 
 #endif  // V8_CODEGEN_IA32_H_