# HG changeset patch
# User thartmann
# Date 1484726069 -3600
# Node ID ff64053e7c5f30fd4ae0960392e5afe395ca157d
# Parent  c11ebe3f754cc5a090d7b1dbe9d7f73b66fa5aed
8172884: TraceOptoPipelining and TraceOptoOutput are broken
Summary: Compiler Control code needs not set the default values for the directive according to the flag value.
Reviewed-by: neliasso, redestad

diff -r c11ebe3f754c -r ff64053e7c5f hotspot/src/share/vm/compiler/compilerDirectives.hpp
--- a/hotspot/src/share/vm/compiler/compilerDirectives.hpp	Tue Jan 17 17:00:22 2017 +0100
+++ b/hotspot/src/share/vm/compiler/compilerDirectives.hpp	Wed Jan 18 08:54:29 2017 +0100
@@ -60,13 +60,12 @@
     cflags(BlockLayoutByFrequency,  bool, BlockLayoutByFrequency,  BlockLayoutByFrequency) \
     cflags(PrintOptoAssembly,       bool, PrintOptoAssembly, PrintOptoAssembly) \
     cflags(PrintIntrinsics,         bool, PrintIntrinsics, PrintIntrinsics) \
-    cflags(TraceOptoPipelining,     bool, false, TraceOptoPipelining) \
-    cflags(TraceOptoOutput,         bool, false, TraceOptoOutput) \
+NOT_PRODUCT(cflags(TraceOptoPipelining, bool, TraceOptoPipelining, TraceOptoPipelining)) \
+NOT_PRODUCT(cflags(TraceOptoOutput,     bool, TraceOptoOutput, TraceOptoOutput)) \
     cflags(TraceSpilling,           bool, TraceSpilling, TraceSpilling) \
     cflags(Vectorize,               bool, false, Vectorize) \
     cflags(VectorizeDebug,         uintx, 0, VectorizeDebug) \
     cflags(CloneMapDebug,           bool, false, CloneMapDebug) \
-    cflags(DoReserveCopyInSuperWordDebug, bool, false, DoReserveCopyInSuperWordDebug) \
     cflags(IGVPrintLevel,           intx, PrintIdealGraphLevel, IGVPrintLevel) \
     cflags(MaxNodeLimit,            intx, MaxNodeLimit, MaxNodeLimit)
 #else
diff -r c11ebe3f754c -r ff64053e7c5f hotspot/src/share/vm/opto/compile.cpp
--- a/hotspot/src/share/vm/opto/compile.cpp	Tue Jan 17 17:00:22 2017 +0100
+++ b/hotspot/src/share/vm/opto/compile.cpp	Wed Jan 18 08:54:29 2017 +0100
@@ -971,7 +971,7 @@
     _java_calls(0),
     _inner_loops(0),
 #ifndef PRODUCT
-    _trace_opto_output(TraceOptoOutput),
+    _trace_opto_output(directive->TraceOptoOutputOption),
     _in_dump_cnt(0),
     _printer(NULL),
 #endif