Reviewers: fschneider, Tobias Burnus,
Message:
Florian: Please take a quick look.
Tobias: Please confirm that this CL fixes the GCC-4.7 warnings you were
seeing.
(Thank you for reporting them!)
Description:
Fix GCC-4.7 warnings
which were introduced in r10700.
Please review this at https://chromiumcodereview.appspot.com/9401005/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files:
M src/compiler.cc
Index: src/compiler.cc
diff --git a/src/compiler.cc b/src/compiler.cc
index
629155e78138f6efc2b7966396f780850f846d43..1a3b4c30623531b8409281a66a2b11254e5601fb
100644
--- a/src/compiler.cc
+++ b/src/compiler.cc
@@ -215,7 +215,6 @@ static bool MakeCrankshaftCode(CompilationInfo* info) {
FLAG_deopt_every_n_times == 0 ? Compiler::kDefaultMaxOptCount : 1000;
if (info->shared_info()->opt_count() > kMaxOptCount) {
info->AbortOptimization();
- Handle<JSFunction> closure = info->closure();
info->shared_info()->DisableOptimization();
// True indicates the compilation pipeline is still going, not
// necessarily that we optimized the code.
@@ -236,7 +235,6 @@ static bool MakeCrankshaftCode(CompilationInfo* info) {
(info->osr_ast_id() != AstNode::kNoNumber &&
scope->num_parameters() + 1 + scope->num_stack_slots() >
locals_limit)) {
info->AbortOptimization();
- Handle<JSFunction> closure = info->closure();
info->shared_info()->DisableOptimization();
// True indicates the compilation pipeline is still going, not
// necessarily that we optimized the code.
@@ -315,7 +313,6 @@ static bool MakeCrankshaftCode(CompilationInfo* info) {
if (!builder.inline_bailout()) {
// Mark the shared code as unoptimizable unless it was an inlined
// function that bailed out.
- Handle<JSFunction> closure = info->closure();
info->shared_info()->DisableOptimization();
}
// True indicates the compilation pipeline is still going, not
necessarily
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
