8205678: ZGC: Remove unused ZAllocationFlags::java_thread()
Reviewed-by: stefank, eosterlund
--- a/src/hotspot/share/gc/z/zAllocationFlags.hpp Wed Jun 27 19:45:05 2018 +0200
+++ b/src/hotspot/share/gc/z/zAllocationFlags.hpp Wed Jun 27 19:45:06 2018 +0200
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2017, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2017, 2018, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -31,31 +31,28 @@
// Allocation flags layout
// -----------------------
//
-// 7 4 3 2 1 0
-// +---+-+-+-+-+-+
-// |000|1|1|1|1|1|
-// +---+-+-+-+-+-+
-// | | | | | |
-// | | | | | * 0-0 Java Thread Flag (1-bit)
-// | | | | |
-// | | | | * 1-1 Worker Thread Flag (1-bit)
-// | | | |
-// | | | * 2-2 Non-Blocking Flag (1-bit)
-// | | |
-// | | * 3-3 Relocation Flag (1-bit)
-// | |
-// | * 4-4 No Reserve Flag (1-bit)
+// 7 3 2 1 0
+// +----+-+-+-+-+
+// |0000|1|1|1|1|
+// +----+-+-+-+-+
+// | | | | |
+// | | | | * 0-0 Worker Thread Flag (1-bit)
+// | | | |
+// | | | * 1-1 Non-Blocking Flag (1-bit)
+// | | |
+// | | * 2-2 Relocation Flag (1-bit)
+// | |
+// | * 3-3 No Reserve Flag (1-bit)
// |
// * 7-5 Unused (3-bits)
//
class ZAllocationFlags {
private:
- typedef ZBitField<uint8_t, bool, 0, 1> field_java_thread;
- typedef ZBitField<uint8_t, bool, 1, 1> field_worker_thread;
- typedef ZBitField<uint8_t, bool, 2, 1> field_non_blocking;
- typedef ZBitField<uint8_t, bool, 3, 1> field_relocation;
- typedef ZBitField<uint8_t, bool, 4, 1> field_no_reserve;
+ typedef ZBitField<uint8_t, bool, 0, 1> field_worker_thread;
+ typedef ZBitField<uint8_t, bool, 1, 1> field_non_blocking;
+ typedef ZBitField<uint8_t, bool, 2, 1> field_relocation;
+ typedef ZBitField<uint8_t, bool, 3, 1> field_no_reserve;
uint8_t _flags;
@@ -63,10 +60,6 @@
ZAllocationFlags() :
_flags(0) {}
- void set_java_thread() {
- _flags |= field_java_thread::encode(true);
- }
-
void set_worker_thread() {
_flags |= field_worker_thread::encode(true);
}
@@ -83,10 +76,6 @@
_flags |= field_no_reserve::encode(true);
}
- bool java_thread() const {
- return field_java_thread::decode(_flags);
- }
-
bool worker_thread() const {
return field_worker_thread::decode(_flags);
}
--- a/src/hotspot/share/gc/z/zObjectAllocator.cpp Wed Jun 27 19:45:05 2018 +0200
+++ b/src/hotspot/share/gc/z/zObjectAllocator.cpp Wed Jun 27 19:45:06 2018 +0200
@@ -186,7 +186,6 @@
assert(ZThread::is_java(), "Must be a Java thread");
ZAllocationFlags flags;
- flags.set_java_thread();
flags.set_no_reserve();
if (!ZStallOnOutOfMemory) {
@@ -205,8 +204,6 @@
if (ZThread::is_worker()) {
flags.set_worker_thread();
- } else if (ZThread::is_java()) {
- flags.set_java_thread();
}
return alloc_object(size, flags);