Skip to content


8296162: [aarch64] Remove unused Address::_is_lval
Browse files Browse the repository at this point in the history
Reviewed-by: chagedorn, aph
  • Loading branch information
Kim Barrett committed Nov 1, 2022
1 parent 15b8b45 commit 2fb64a4
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
1 change: 0 additions & 1 deletion src/hotspot/cpu/aarch64/assembler_aarch64.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,6 @@ void Assembler::add_sub_immediate(Instruction_aarch64 &current_insn,
#undef starti

Address::Address(address target, relocInfo::relocType rtype) : _mode(literal){
_is_lval = false;
_target = target;
switch (rtype) {
case relocInfo::oop_type:
Expand Down
6 changes: 0 additions & 6 deletions src/hotspot/cpu/aarch64/assembler_aarch64.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -372,11 +372,6 @@ class Address {

RelocationHolder _rspec;

// Typically we use AddressLiterals we want to use their rval
// However in some situations we want the lval (effect address) of
// the item. We provide a special factory for making those lvals.
bool _is_lval;

// If the target is far we'll need to load the ea of this to a
// register to reach it. Otherwise if near we can do PC-relative
// addressing.
Expand Down Expand Up @@ -405,7 +400,6 @@ class Address {
Address(address target, RelocationHolder const& rspec)
: _mode(literal),
_target(target) { }
Address(address target, relocInfo::relocType rtype = relocInfo::external_word_type);
Address(Register base, RegisterOrConstant index, extend ext = lsl())
Expand Down

1 comment on commit 2fb64a4

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.