8364723: Sort share/interpreter includes

Reviewed-by: shade, ayang

src/hotspot/share/interpreter/abstractInterpreter.cpp

@@ -22,23 +22,22 @@
  *
  */
 
-#include "asm/macroAssembler.hpp"
 #include "asm/macroAssembler.inline.hpp"
 #include "cds/metaspaceShared.hpp"
 #include "compiler/disassembler.hpp"
 #include "interpreter/bytecodeHistogram.hpp"
 #include "interpreter/bytecodeStream.hpp"
+#include "interpreter/interp_masm.hpp"
 #include "interpreter/interpreter.hpp"
 #include "interpreter/interpreterRuntime.hpp"
-#include "interpreter/interp_masm.hpp"
 #include "interpreter/templateTable.hpp"
 #include "memory/allocation.inline.hpp"
 #include "memory/resourceArea.hpp"
 #include "oops/arrayOop.hpp"
 #include "oops/constantPool.inline.hpp"
 #include "oops/cpCache.inline.hpp"
-#include "oops/methodData.hpp"
 #include "oops/method.inline.hpp"
+#include "oops/methodData.hpp"
 #include "oops/oop.inline.hpp"
 #include "prims/jvmtiExport.hpp"
 #include "prims/methodHandles.hpp"

src/hotspot/share/interpreter/bytecodeStream.cpp

@@ -22,8 +22,8 @@
  *
  */
 
-#include "interpreter/bytecodeStream.hpp"
 #include "interpreter/bytecodes.hpp"
+#include "interpreter/bytecodeStream.hpp"
 #include "runtime/handles.inline.hpp"
 
 Bytecodes::Code RawBytecodeStream::raw_next_special(Bytecodes::Code code) {

src/hotspot/share/interpreter/bytecodeTracer.cpp

@@ -25,14 +25,14 @@
 #include "classfile/classPrinter.hpp"
 #include "classfile/javaClasses.inline.hpp"
 #include "interpreter/bytecodeHistogram.hpp"
+#include "interpreter/bytecodes.hpp"
 #include "interpreter/bytecodeStream.hpp"
 #include "interpreter/bytecodeTracer.hpp"
-#include "interpreter/bytecodes.hpp"
 #include "interpreter/interpreter.hpp"
 #include "memory/resourceArea.hpp"
 #include "oops/constantPool.inline.hpp"
-#include "oops/methodData.hpp"
 #include "oops/method.hpp"
+#include "oops/methodData.hpp"
 #include "runtime/atomic.hpp"
 #include "runtime/handles.inline.hpp"
 #include "runtime/mutexLocker.hpp"

src/hotspot/share/interpreter/bytecodeUtils.cpp

@@ -28,8 +28,8 @@
 #include "gc/shared/gcLocker.hpp"
 #include "interpreter/bytecodeUtils.hpp"
 #include "memory/resourceArea.hpp"
-#include "runtime/signature.hpp"
 #include "runtime/safepointVerifiers.hpp"
+#include "runtime/signature.hpp"
 #include "utilities/events.hpp"
 #include "utilities/ostream.hpp"
 

src/hotspot/share/interpreter/interpreter.cpp

@@ -22,18 +22,17 @@
  *
  */
 
-#include "asm/macroAssembler.hpp"
 #include "asm/macroAssembler.inline.hpp"
 #include "compiler/disassembler.hpp"
+#include "interpreter/interp_masm.hpp"
 #include "interpreter/interpreter.hpp"
 #include "interpreter/interpreterRuntime.hpp"
-#include "interpreter/interp_masm.hpp"
 #include "interpreter/templateTable.hpp"
 #include "memory/allocation.inline.hpp"
 #include "memory/resourceArea.hpp"
 #include "oops/arrayOop.hpp"
-#include "oops/methodData.hpp"
 #include "oops/method.hpp"
+#include "oops/methodData.hpp"
 #include "oops/oop.inline.hpp"
 #include "prims/forte.hpp"
 #include "prims/jvmtiExport.hpp"

src/hotspot/share/interpreter/interpreterRuntime.cpp

@@ -46,8 +46,8 @@
 #include "oops/cpCache.inline.hpp"
 #include "oops/instanceKlass.inline.hpp"
 #include "oops/klass.inline.hpp"
-#include "oops/methodData.hpp"
 #include "oops/method.inline.hpp"
+#include "oops/methodData.hpp"
 #include "oops/objArrayKlass.hpp"
 #include "oops/objArrayOop.inline.hpp"
 #include "oops/oop.inline.hpp"

src/hotspot/share/interpreter/templateInterpreter.cpp

@@ -22,9 +22,9 @@
  *
  */
 
+#include "interpreter/interp_masm.hpp"
 #include "interpreter/interpreter.hpp"
 #include "interpreter/interpreterRuntime.hpp"
-#include "interpreter/interp_masm.hpp"
 #include "interpreter/templateInterpreter.hpp"
 #include "interpreter/templateInterpreterGenerator.hpp"
 #include "interpreter/templateTable.hpp"

src/hotspot/share/interpreter/templateInterpreterGenerator.cpp

@@ -23,9 +23,9 @@
  */
 
 #include "compiler/disassembler.hpp"
+#include "interpreter/interp_masm.hpp"
 #include "interpreter/interpreter.hpp"
 #include "interpreter/interpreterRuntime.hpp"
-#include "interpreter/interp_masm.hpp"
 #include "interpreter/templateInterpreter.hpp"
 #include "interpreter/templateInterpreterGenerator.hpp"
 #include "interpreter/templateTable.hpp"

src/hotspot/share/interpreter/zero/bytecodeInterpreter.cpp

@@ -29,9 +29,9 @@
 #include "gc/shared/threadLocalAllocBuffer.inline.hpp"
 #include "gc/shared/tlab_globals.hpp"
 #include "interpreter/bytecodeHistogram.hpp"
-#include "interpreter/zero/bytecodeInterpreter.inline.hpp"
 #include "interpreter/interpreter.hpp"
 #include "interpreter/interpreterRuntime.hpp"
+#include "interpreter/zero/bytecodeInterpreter.inline.hpp"
 #include "jvm_io.h"
 #include "logging/log.hpp"
 #include "memory/resourceArea.hpp"

src/hotspot/share/interpreter/zero/bytecodeInterpreter.inline.hpp

@@ -27,6 +27,7 @@
 
 #include "interpreter/zero/bytecodeInterpreter.hpp"
 
+#include "bytecodeInterpreter_zero.inline.hpp"
 #include "runtime/stubRoutines.hpp"
 #include "utilities/macros.hpp"
 
@@ -42,6 +43,4 @@
 #define VERIFY_OOP(o)
 #endif
 
-# include "bytecodeInterpreter_zero.inline.hpp"
-
 #endif // SHARE_INTERPRETER_BYTECODEINTERPRETER_INLINE_HPP

src/hotspot/share/interpreter/zero/zeroInterpreterGenerator.hpp

@@ -29,7 +29,6 @@
 // of the Zero interpreter generator.
 
 # include "entry_zero.hpp"
-// # include "interpreter/interp_masm.hpp"
 
 class ZeroInterpreterGenerator: public AbstractInterpreterGenerator {
 

test/hotspot/jtreg/sources/TestIncludesAreSorted.java

@@ -50,6 +50,7 @@ public class TestIncludesAreSorted {
                     "share/classfile",
                     "share/code",
                     "share/compiler",
+                    "share/interpreter",
                     "share/jvmci",
                     "share/libadt",
                     "share/metaprogramming",