8172884: TraceOptoPipelining and TraceOptoOutput are broken
authorthartmann
Wed, 18 Jan 2017 08:54:29 +0100
changeset 43456 ff64053e7c5f
parent 43454 c11ebe3f754c
child 43457 617ed26c48d2
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
hotspot/src/share/vm/compiler/compilerDirectives.hpp
hotspot/src/share/vm/opto/compile.cpp
--- 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
--- 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