Print reason for disabling optimization. Kill --trace-bailout flag.

src/lithium.cc

Index: src/lithium.cc
diff --git a/src/lithium.cc b/src/lithium.cc
index f19ba006fa852fd9b681686d018930b7f662873c..a9cd023dc4823801ddcc68e0da6c2027ecfdf2a2 100644
--- a/src/lithium.cc
+++ b/src/lithium.cc
@@ -390,15 +390,13 @@ Representation LChunk::LookupLiteralRepresentation(
 }
 
 
-LChunk* LChunk::NewChunk(HGraph* graph) {
+LChunk* LChunk::NewChunk(HGraph* graph, CompilationInfo* info) {

[Jakob Kummerow, 2012/08/27 14:26:02]

Instead of passing in |info|, can't you just use graph->info() (as line 403 does
anyway)?

[Sven Panne, 2012/08/28 07:17:08]

Good point, I missed that one after refactoring back and forth. Not needing
another argument is a good sign that CompilationInfo is indeed the right place
to keep the bailout reason. Done.

   NoHandleAllocation no_handles;
   AssertNoAllocation no_gc;
 
   int values = graph->GetMaximumValueID();
   if (values > LUnallocated::kMaxVirtualRegisters) {
-    if (FLAG_trace_bailout) {
-      PrintF("Not enough virtual registers for (values).\n");
-    }
+    info->set_bailout_reason("not enough virtual registers for values");
     return NULL;
   }
   LAllocator allocator(values, graph);
@@ -407,9 +405,7 @@ LChunk* LChunk::NewChunk(HGraph* graph) {
   if (chunk == NULL) return NULL;
 
   if (!allocator.Allocate(chunk)) {
-    if (FLAG_trace_bailout) {
-      PrintF("Not enough virtual registers (regalloc).\n");
-    }
+    info->set_bailout_reason("not enough virtual registers (regalloc)");
     return NULL;
   }