changeset 43456:ff64053e7c5f

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
author thartmann
date Wed, 18 Jan 2017 08:54:29 +0100
parents c11ebe3f754c
children 617ed26c48d2
files hotspot/src/share/vm/compiler/compilerDirectives.hpp hotspot/src/share/vm/opto/compile.cpp
diffstat 2 files changed, 3 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- 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