Skip to content

Commit

Permalink
8257418: C2: Rename barrier data member in MemNode and LoadStoreNode
Browse files Browse the repository at this point in the history
Reviewed-by: vlivanov
  • Loading branch information
pliden committed Nov 30, 2020
1 parent c071960 commit e3abe51
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 10 deletions.
2 changes: 1 addition & 1 deletion src/hotspot/share/opto/memnode.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2956,7 +2956,7 @@ LoadStoreNode::LoadStoreNode( Node *c, Node *mem, Node *adr, Node *val, const Ty
: Node(required),
_type(rt),
_adr_type(at),
_barrier(0)
_barrier_data(0)
{
init_req(MemNode::Control, c );
init_req(MemNode::Memory , mem);
Expand Down
18 changes: 9 additions & 9 deletions src/hotspot/share/opto/memnode.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ class MemNode : public Node {
bool _unaligned_access; // Unaligned access from unsafe
bool _mismatched_access; // Mismatched access from unsafe: byte read in integer array for instance
bool _unsafe_access; // Access of unsafe origin.
uint8_t _barrier; // Bit field with barrier information
uint8_t _barrier_data; // Bit field with barrier information

protected:
#ifdef ASSERT
Expand All @@ -69,7 +69,7 @@ class MemNode : public Node {
_unaligned_access(false),
_mismatched_access(false),
_unsafe_access(false),
_barrier(0) {
_barrier_data(0) {
init_class_id(Class_Mem);
debug_only(_adr_type=at; adr_type();)
}
Expand All @@ -78,7 +78,7 @@ class MemNode : public Node {
_unaligned_access(false),
_mismatched_access(false),
_unsafe_access(false),
_barrier(0) {
_barrier_data(0) {
init_class_id(Class_Mem);
debug_only(_adr_type=at; adr_type();)
}
Expand All @@ -87,7 +87,7 @@ class MemNode : public Node {
_unaligned_access(false),
_mismatched_access(false),
_unsafe_access(false),
_barrier(0) {
_barrier_data(0) {
init_class_id(Class_Mem);
debug_only(_adr_type=at; adr_type();)
}
Expand Down Expand Up @@ -140,8 +140,8 @@ class MemNode : public Node {
#endif
}

uint8_t barrier_data() { return _barrier; }
void set_barrier_data(uint8_t barrier_data) { _barrier = barrier_data; }
uint8_t barrier_data() { return _barrier_data; }
void set_barrier_data(uint8_t barrier_data) { _barrier_data = barrier_data; }

// Search through memory states which precede this node (load or store).
// Look for an exact match for the address, with no intervening
Expand Down Expand Up @@ -839,7 +839,7 @@ class LoadStoreNode : public Node {
private:
const Type* const _type; // What kind of value is loaded?
const TypePtr* _adr_type; // What kind of memory is being addressed?
uint8_t _barrier; // Bit field with barrier information
uint8_t _barrier_data; // Bit field with barrier information
virtual uint size_of() const; // Size is bigger
public:
LoadStoreNode( Node *c, Node *mem, Node *adr, Node *val, const TypePtr* at, const Type* rt, uint required );
Expand All @@ -853,8 +853,8 @@ class LoadStoreNode : public Node {
bool result_not_used() const;
MemBarNode* trailing_membar() const;

uint8_t barrier_data() { return _barrier; }
void set_barrier_data(uint8_t barrier_data) { _barrier = barrier_data; }
uint8_t barrier_data() { return _barrier_data; }
void set_barrier_data(uint8_t barrier_data) { _barrier_data = barrier_data; }
};

class LoadStoreConditionalNode : public LoadStoreNode {
Expand Down

1 comment on commit e3abe51

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.