diff options
Diffstat (limited to 'recipes-core/openjdk/patches-openjdk-8/1007-hotspot-Turn-on-the-Wreturn-type-warning.patch')
-rw-r--r-- | recipes-core/openjdk/patches-openjdk-8/1007-hotspot-Turn-on-the-Wreturn-type-warning.patch | 72 |
1 files changed, 37 insertions, 35 deletions
diff --git a/recipes-core/openjdk/patches-openjdk-8/1007-hotspot-Turn-on-the-Wreturn-type-warning.patch b/recipes-core/openjdk/patches-openjdk-8/1007-hotspot-Turn-on-the-Wreturn-type-warning.patch index 0dc09b6..b1b59c3 100644 --- a/recipes-core/openjdk/patches-openjdk-8/1007-hotspot-Turn-on-the-Wreturn-type-warning.patch +++ b/recipes-core/openjdk/patches-openjdk-8/1007-hotspot-Turn-on-the-Wreturn-type-warning.patch | |||
@@ -1,7 +1,11 @@ | |||
1 | From 024e0d522f205ca8659fce60ca4be68b9e3e08db Mon Sep 17 00:00:00 2001 | 1 | From 6758bf27e31110dc5d4c7237476c823ff05454a2 Mon Sep 17 00:00:00 2001 |
2 | From: Richard Leitner <richard.leitner@skidata.com> | 2 | From: Richard Leitner <richard.leitner@skidata.com> |
3 | Date: Thu, 22 Oct 2020 09:41:07 +0200 | 3 | Date: Thu, 22 Oct 2020 09:41:07 +0200 |
4 | Subject: [PATCH 1007/1013] hotspot: Turn on the -Wreturn-type warning | 4 | Subject: [PATCH] hotspot: Turn on the -Wreturn-type warning |
5 | MIME-Version: 1.0 | ||
6 | Content-Type: text/plain; charset=UTF-8 | ||
7 | Content-Transfer-Encoding: 8bit | ||
8 | |||
5 | MIME-Version: 1.0 | 9 | MIME-Version: 1.0 |
6 | Content-Type: text/plain; charset=UTF-8 | 10 | Content-Type: text/plain; charset=UTF-8 |
7 | Content-Transfer-Encoding: 8bit | 11 | Content-Transfer-Encoding: 8bit |
@@ -19,27 +23,28 @@ Upstream-Status: Backport | |||
19 | 23 | ||
20 | Signed-off-by: André Draszik <andre.draszik@jci.com> | 24 | Signed-off-by: André Draszik <andre.draszik@jci.com> |
21 | Signed-off-by: Richard Leitner <richard.leitner@skidata.com> | 25 | Signed-off-by: Richard Leitner <richard.leitner@skidata.com> |
26 | |||
22 | --- | 27 | --- |
23 | make/linux/makefiles/gcc.make | 2 +- | 28 | hotspot/make/linux/makefiles/gcc.make | 2 +- |
24 | src/cpu/x86/vm/x86_32.ad | 1 + | 29 | hotspot/src/cpu/x86/vm/x86_32.ad | 1 + |
25 | src/os_cpu/linux_x86/vm/os_linux_x86.cpp | 1 + | 30 | hotspot/src/os_cpu/linux_x86/vm/os_linux_x86.cpp | 1 + |
26 | src/share/vm/classfile/defaultMethods.cpp | 4 ++-- | 31 | hotspot/src/share/vm/classfile/defaultMethods.cpp | 4 ++-- |
27 | src/share/vm/classfile/symbolTable.cpp | 4 ++-- | 32 | hotspot/src/share/vm/classfile/symbolTable.cpp | 4 ++-- |
28 | src/share/vm/classfile/systemDictionary.cpp | 6 +++--- | 33 | hotspot/src/share/vm/classfile/systemDictionary.cpp | 6 +++--- |
29 | src/share/vm/memory/heapInspection.hpp | 2 +- | 34 | hotspot/src/share/vm/memory/heapInspection.hpp | 2 +- |
30 | src/share/vm/memory/metaspaceShared.hpp | 2 +- | 35 | hotspot/src/share/vm/memory/metaspaceShared.hpp | 2 +- |
31 | src/share/vm/oops/constantPool.hpp | 2 +- | 36 | hotspot/src/share/vm/oops/constantPool.hpp | 2 +- |
32 | src/share/vm/prims/jvm.cpp | 2 +- | 37 | hotspot/src/share/vm/prims/jvm.cpp | 2 +- |
33 | src/share/vm/runtime/reflection.cpp | 2 +- | 38 | hotspot/src/share/vm/runtime/reflection.cpp | 2 +- |
34 | src/share/vm/runtime/sharedRuntime.cpp | 2 +- | 39 | hotspot/src/share/vm/runtime/sharedRuntime.cpp | 2 +- |
35 | src/share/vm/services/memTracker.hpp | 2 +- | 40 | hotspot/src/share/vm/services/memTracker.hpp | 2 +- |
36 | 13 files changed, 17 insertions(+), 15 deletions(-) | 41 | 13 files changed, 17 insertions(+), 15 deletions(-) |
37 | 42 | ||
38 | diff --git a/hotspot/make/linux/makefiles/gcc.make b/hotspot/make/linux/makefiles/gcc.make | 43 | diff --git a/hotspot/make/linux/makefiles/gcc.make b/hotspot/make/linux/makefiles/gcc.make |
39 | index ac44b6837..7a5162a06 100644 | 44 | index 36b3c8ba..cd9511e5 100644 |
40 | --- a/hotspot/make/linux/makefiles/gcc.make | 45 | --- a/hotspot/make/linux/makefiles/gcc.make |
41 | +++ b/hotspot/make/linux/makefiles/gcc.make | 46 | +++ b/hotspot/make/linux/makefiles/gcc.make |
42 | @@ -212,7 +212,7 @@ ifeq ($(USE_CLANG), true) | 47 | @@ -211,7 +211,7 @@ ifeq ($(USE_CLANG), true) |
43 | WARNINGS_ARE_ERRORS += -Wno-return-type -Wno-empty-body | 48 | WARNINGS_ARE_ERRORS += -Wno-return-type -Wno-empty-body |
44 | endif | 49 | endif |
45 | 50 | ||
@@ -49,7 +54,7 @@ index ac44b6837..7a5162a06 100644 | |||
49 | ifeq ($(USE_CLANG),) | 54 | ifeq ($(USE_CLANG),) |
50 | # Since GCC 4.3, -Wconversion has changed its meanings to warn these implicit | 55 | # Since GCC 4.3, -Wconversion has changed its meanings to warn these implicit |
51 | diff --git a/hotspot/src/cpu/x86/vm/x86_32.ad b/hotspot/src/cpu/x86/vm/x86_32.ad | 56 | diff --git a/hotspot/src/cpu/x86/vm/x86_32.ad b/hotspot/src/cpu/x86/vm/x86_32.ad |
52 | index f42d1a288..c8f4ee161 100644 | 57 | index f42d1a28..c8f4ee16 100644 |
53 | --- a/hotspot/src/cpu/x86/vm/x86_32.ad | 58 | --- a/hotspot/src/cpu/x86/vm/x86_32.ad |
54 | +++ b/hotspot/src/cpu/x86/vm/x86_32.ad | 59 | +++ b/hotspot/src/cpu/x86/vm/x86_32.ad |
55 | @@ -1250,6 +1250,7 @@ uint MachSpillCopyNode::implementation( CodeBuffer *cbuf, PhaseRegAlloc *ra_, bo | 60 | @@ -1250,6 +1250,7 @@ uint MachSpillCopyNode::implementation( CodeBuffer *cbuf, PhaseRegAlloc *ra_, bo |
@@ -61,7 +66,7 @@ index f42d1a288..c8f4ee161 100644 | |||
61 | 66 | ||
62 | #ifndef PRODUCT | 67 | #ifndef PRODUCT |
63 | diff --git a/hotspot/src/os_cpu/linux_x86/vm/os_linux_x86.cpp b/hotspot/src/os_cpu/linux_x86/vm/os_linux_x86.cpp | 68 | diff --git a/hotspot/src/os_cpu/linux_x86/vm/os_linux_x86.cpp b/hotspot/src/os_cpu/linux_x86/vm/os_linux_x86.cpp |
64 | index c35d8677f..65c3165ca 100644 | 69 | index c35d8677..65c3165c 100644 |
65 | --- a/hotspot/src/os_cpu/linux_x86/vm/os_linux_x86.cpp | 70 | --- a/hotspot/src/os_cpu/linux_x86/vm/os_linux_x86.cpp |
66 | +++ b/hotspot/src/os_cpu/linux_x86/vm/os_linux_x86.cpp | 71 | +++ b/hotspot/src/os_cpu/linux_x86/vm/os_linux_x86.cpp |
67 | @@ -541,6 +541,7 @@ JVM_handle_linux_signal(int sig, | 72 | @@ -541,6 +541,7 @@ JVM_handle_linux_signal(int sig, |
@@ -73,7 +78,7 @@ index c35d8677f..65c3165ca 100644 | |||
73 | 78 | ||
74 | void os::Linux::init_thread_fpu_state(void) { | 79 | void os::Linux::init_thread_fpu_state(void) { |
75 | diff --git a/hotspot/src/share/vm/classfile/defaultMethods.cpp b/hotspot/src/share/vm/classfile/defaultMethods.cpp | 80 | diff --git a/hotspot/src/share/vm/classfile/defaultMethods.cpp b/hotspot/src/share/vm/classfile/defaultMethods.cpp |
76 | index 4b4b4e250..196622aed 100644 | 81 | index 4b4b4e25..196622ae 100644 |
77 | --- a/hotspot/src/share/vm/classfile/defaultMethods.cpp | 82 | --- a/hotspot/src/share/vm/classfile/defaultMethods.cpp |
78 | +++ b/hotspot/src/share/vm/classfile/defaultMethods.cpp | 83 | +++ b/hotspot/src/share/vm/classfile/defaultMethods.cpp |
79 | @@ -506,7 +506,7 @@ Symbol* MethodFamily::generate_method_message(Symbol *klass_name, Method* method | 84 | @@ -506,7 +506,7 @@ Symbol* MethodFamily::generate_method_message(Symbol *klass_name, Method* method |
@@ -95,7 +100,7 @@ index 4b4b4e250..196622aed 100644 | |||
95 | 100 | ||
96 | 101 | ||
97 | diff --git a/hotspot/src/share/vm/classfile/symbolTable.cpp b/hotspot/src/share/vm/classfile/symbolTable.cpp | 102 | diff --git a/hotspot/src/share/vm/classfile/symbolTable.cpp b/hotspot/src/share/vm/classfile/symbolTable.cpp |
98 | index ec97077b7..2621a7d62 100644 | 103 | index 83369cbc..8dd4e6b2 100644 |
99 | --- a/hotspot/src/share/vm/classfile/symbolTable.cpp | 104 | --- a/hotspot/src/share/vm/classfile/symbolTable.cpp |
100 | +++ b/hotspot/src/share/vm/classfile/symbolTable.cpp | 105 | +++ b/hotspot/src/share/vm/classfile/symbolTable.cpp |
101 | @@ -249,7 +249,7 @@ Symbol* SymbolTable::lookup(const char* name, int len, TRAPS) { | 106 | @@ -249,7 +249,7 @@ Symbol* SymbolTable::lookup(const char* name, int len, TRAPS) { |
@@ -117,7 +122,7 @@ index ec97077b7..2621a7d62 100644 | |||
117 | 122 | ||
118 | Symbol* SymbolTable::lookup_only(const char* name, int len, | 123 | Symbol* SymbolTable::lookup_only(const char* name, int len, |
119 | diff --git a/hotspot/src/share/vm/classfile/systemDictionary.cpp b/hotspot/src/share/vm/classfile/systemDictionary.cpp | 124 | diff --git a/hotspot/src/share/vm/classfile/systemDictionary.cpp b/hotspot/src/share/vm/classfile/systemDictionary.cpp |
120 | index 5911d94f5..df4d56f6f 100644 | 125 | index 4541e815..98412e1e 100644 |
121 | --- a/hotspot/src/share/vm/classfile/systemDictionary.cpp | 126 | --- a/hotspot/src/share/vm/classfile/systemDictionary.cpp |
122 | +++ b/hotspot/src/share/vm/classfile/systemDictionary.cpp | 127 | +++ b/hotspot/src/share/vm/classfile/systemDictionary.cpp |
123 | @@ -236,15 +236,15 @@ Klass* SystemDictionary::resolve_or_null(Symbol* class_name, Handle class_loader | 128 | @@ -236,15 +236,15 @@ Klass* SystemDictionary::resolve_or_null(Symbol* class_name, Handle class_loader |
@@ -140,7 +145,7 @@ index 5911d94f5..df4d56f6f 100644 | |||
140 | } | 145 | } |
141 | 146 | ||
142 | diff --git a/hotspot/src/share/vm/memory/heapInspection.hpp b/hotspot/src/share/vm/memory/heapInspection.hpp | 147 | diff --git a/hotspot/src/share/vm/memory/heapInspection.hpp b/hotspot/src/share/vm/memory/heapInspection.hpp |
143 | index 09ee6602e..c5fec15c3 100644 | 148 | index 09ee6602..c5fec15c 100644 |
144 | --- a/hotspot/src/share/vm/memory/heapInspection.hpp | 149 | --- a/hotspot/src/share/vm/memory/heapInspection.hpp |
145 | +++ b/hotspot/src/share/vm/memory/heapInspection.hpp | 150 | +++ b/hotspot/src/share/vm/memory/heapInspection.hpp |
146 | @@ -367,7 +367,7 @@ class HeapInspection : public StackObj { | 151 | @@ -367,7 +367,7 @@ class HeapInspection : public StackObj { |
@@ -153,7 +158,7 @@ index 09ee6602e..c5fec15c3 100644 | |||
153 | private: | 158 | private: |
154 | void iterate_over_heap(KlassInfoTable* cit, BoolObjectClosure* filter = NULL); | 159 | void iterate_over_heap(KlassInfoTable* cit, BoolObjectClosure* filter = NULL); |
155 | diff --git a/hotspot/src/share/vm/memory/metaspaceShared.hpp b/hotspot/src/share/vm/memory/metaspaceShared.hpp | 160 | diff --git a/hotspot/src/share/vm/memory/metaspaceShared.hpp b/hotspot/src/share/vm/memory/metaspaceShared.hpp |
156 | index 2f3abae6a..d58ebecb2 100644 | 161 | index 2f3abae6..d58ebecb 100644 |
157 | --- a/hotspot/src/share/vm/memory/metaspaceShared.hpp | 162 | --- a/hotspot/src/share/vm/memory/metaspaceShared.hpp |
158 | +++ b/hotspot/src/share/vm/memory/metaspaceShared.hpp | 163 | +++ b/hotspot/src/share/vm/memory/metaspaceShared.hpp |
159 | @@ -93,7 +93,7 @@ class MetaspaceShared : AllStatic { | 164 | @@ -93,7 +93,7 @@ class MetaspaceShared : AllStatic { |
@@ -166,10 +171,10 @@ index 2f3abae6a..d58ebecb2 100644 | |||
166 | static ReservedSpace* shared_rs() { | 171 | static ReservedSpace* shared_rs() { |
167 | CDS_ONLY(return _shared_rs); | 172 | CDS_ONLY(return _shared_rs); |
168 | diff --git a/hotspot/src/share/vm/oops/constantPool.hpp b/hotspot/src/share/vm/oops/constantPool.hpp | 173 | diff --git a/hotspot/src/share/vm/oops/constantPool.hpp b/hotspot/src/share/vm/oops/constantPool.hpp |
169 | index 124c970e2..dae574c09 100644 | 174 | index 68435471..fad4a92c 100644 |
170 | --- a/hotspot/src/share/vm/oops/constantPool.hpp | 175 | --- a/hotspot/src/share/vm/oops/constantPool.hpp |
171 | +++ b/hotspot/src/share/vm/oops/constantPool.hpp | 176 | +++ b/hotspot/src/share/vm/oops/constantPool.hpp |
172 | @@ -353,7 +353,7 @@ class ConstantPool : public Metadata { | 177 | @@ -350,7 +350,7 @@ class ConstantPool : public Metadata { |
173 | 178 | ||
174 | Klass* klass_at(int which, TRAPS) { | 179 | Klass* klass_at(int which, TRAPS) { |
175 | constantPoolHandle h_this(THREAD, this); | 180 | constantPoolHandle h_this(THREAD, this); |
@@ -177,9 +182,9 @@ index 124c970e2..dae574c09 100644 | |||
177 | + return klass_at_impl(h_this, which, THREAD); | 182 | + return klass_at_impl(h_this, which, THREAD); |
178 | } | 183 | } |
179 | 184 | ||
180 | Symbol* klass_name_at(int which); // Returns the name, w/o resolving. | 185 | Symbol* klass_name_at(int which) const; // Returns the name, w/o resolving. |
181 | diff --git a/hotspot/src/share/vm/prims/jvm.cpp b/hotspot/src/share/vm/prims/jvm.cpp | 186 | diff --git a/hotspot/src/share/vm/prims/jvm.cpp b/hotspot/src/share/vm/prims/jvm.cpp |
182 | index 6f2221c41..daa69f89a 100644 | 187 | index 6f2221c4..daa69f89 100644 |
183 | --- a/hotspot/src/share/vm/prims/jvm.cpp | 188 | --- a/hotspot/src/share/vm/prims/jvm.cpp |
184 | +++ b/hotspot/src/share/vm/prims/jvm.cpp | 189 | +++ b/hotspot/src/share/vm/prims/jvm.cpp |
185 | @@ -4368,7 +4368,7 @@ JVM_ENTRY(jlong,JVM_DTraceActivate( | 190 | @@ -4368,7 +4368,7 @@ JVM_ENTRY(jlong,JVM_DTraceActivate( |
@@ -192,7 +197,7 @@ index 6f2221c41..daa69f89a 100644 | |||
192 | 197 | ||
193 | JVM_ENTRY(jboolean,JVM_DTraceIsProbeEnabled(JNIEnv* env, jmethodID method)) | 198 | JVM_ENTRY(jboolean,JVM_DTraceIsProbeEnabled(JNIEnv* env, jmethodID method)) |
194 | diff --git a/hotspot/src/share/vm/runtime/reflection.cpp b/hotspot/src/share/vm/runtime/reflection.cpp | 199 | diff --git a/hotspot/src/share/vm/runtime/reflection.cpp b/hotspot/src/share/vm/runtime/reflection.cpp |
195 | index d9fddbe47..a5f737935 100644 | 200 | index 4b39d8cc..9cd7b0dc 100644 |
196 | --- a/hotspot/src/share/vm/runtime/reflection.cpp | 201 | --- a/hotspot/src/share/vm/runtime/reflection.cpp |
197 | +++ b/hotspot/src/share/vm/runtime/reflection.cpp | 202 | +++ b/hotspot/src/share/vm/runtime/reflection.cpp |
198 | @@ -1093,7 +1093,7 @@ oop Reflection::invoke(instanceKlassHandle klass, methodHandle reflected_method, | 203 | @@ -1093,7 +1093,7 @@ oop Reflection::invoke(instanceKlassHandle klass, methodHandle reflected_method, |
@@ -205,10 +210,10 @@ index d9fddbe47..a5f737935 100644 | |||
205 | } | 210 | } |
206 | 211 | ||
207 | diff --git a/hotspot/src/share/vm/runtime/sharedRuntime.cpp b/hotspot/src/share/vm/runtime/sharedRuntime.cpp | 212 | diff --git a/hotspot/src/share/vm/runtime/sharedRuntime.cpp b/hotspot/src/share/vm/runtime/sharedRuntime.cpp |
208 | index 013aa7a23..df4cde963 100644 | 213 | index d5bea29c..50578ad6 100644 |
209 | --- a/hotspot/src/share/vm/runtime/sharedRuntime.cpp | 214 | --- a/hotspot/src/share/vm/runtime/sharedRuntime.cpp |
210 | +++ b/hotspot/src/share/vm/runtime/sharedRuntime.cpp | 215 | +++ b/hotspot/src/share/vm/runtime/sharedRuntime.cpp |
211 | @@ -1045,7 +1045,7 @@ Handle SharedRuntime::find_callee_info(JavaThread* thread, Bytecodes::Code& bc, | 216 | @@ -1041,7 +1041,7 @@ Handle SharedRuntime::find_callee_info(JavaThread* thread, Bytecodes::Code& bc, |
212 | // last java frame on stack (which includes native call frames) | 217 | // last java frame on stack (which includes native call frames) |
213 | vframeStream vfst(thread, true); // Do not skip and javaCalls | 218 | vframeStream vfst(thread, true); // Do not skip and javaCalls |
214 | 219 | ||
@@ -218,7 +223,7 @@ index 013aa7a23..df4cde963 100644 | |||
218 | 223 | ||
219 | 224 | ||
220 | diff --git a/hotspot/src/share/vm/services/memTracker.hpp b/hotspot/src/share/vm/services/memTracker.hpp | 225 | diff --git a/hotspot/src/share/vm/services/memTracker.hpp b/hotspot/src/share/vm/services/memTracker.hpp |
221 | index 8ea859ddb..535147f8c 100644 | 226 | index 8ea859dd..535147f8 100644 |
222 | --- a/hotspot/src/share/vm/services/memTracker.hpp | 227 | --- a/hotspot/src/share/vm/services/memTracker.hpp |
223 | +++ b/hotspot/src/share/vm/services/memTracker.hpp | 228 | +++ b/hotspot/src/share/vm/services/memTracker.hpp |
224 | @@ -64,7 +64,7 @@ class MemTracker : AllStatic { | 229 | @@ -64,7 +64,7 @@ class MemTracker : AllStatic { |
@@ -230,6 +235,3 @@ index 8ea859ddb..535147f8c 100644 | |||
230 | static inline void record_virtual_memory_type(void* addr, MEMFLAGS flag) { } | 235 | static inline void record_virtual_memory_type(void* addr, MEMFLAGS flag) { } |
231 | static inline void record_thread_stack(void* addr, size_t size) { } | 236 | static inline void record_thread_stack(void* addr, size_t size) { } |
232 | static inline void release_thread_stack(void* addr, size_t size) { } | 237 | static inline void release_thread_stack(void* addr, size_t size) { } |
233 | -- | ||
234 | 2.26.2 | ||
235 | |||