src/hotspot/share/asm/assembler.cpp
author rkennke
Wed, 14 Nov 2018 18:57:47 +0100
changeset 52559 f642ede2eb07
parent 52462 4ad404da0088
child 54780 f8d182aedc92
permissions -rw-r--r--
8213795: Force explicit null check on patching placeholder offset Reviewed-by: kvn, adinn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 41685
diff changeset
     2
 * Copyright (c) 1997, 2017, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5403
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5403
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5403
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    25
#include "precompiled.hpp"
35201
996db89f378e 8139864: Improve handling of stack protection zones.
goetz
parents: 33105
diff changeset
    26
#include "asm/codeBuffer.hpp"
14626
0cf4eccf130f 8003240: x86: move MacroAssembler into separate file
twisti
parents: 14625
diff changeset
    27
#include "asm/macroAssembler.hpp"
0cf4eccf130f 8003240: x86: move MacroAssembler into separate file
twisti
parents: 14625
diff changeset
    28
#include "asm/macroAssembler.inline.hpp"
52462
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
    29
#include "gc/shared/collectedHeap.hpp"
40655
9f644073d3a0 8157907: Incorrect inclusion of atomic.hpp instead of atomic.inline.hpp
dholmes
parents: 35201
diff changeset
    30
#include "runtime/atomic.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    31
#include "runtime/icache.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    32
#include "runtime/os.hpp"
35201
996db89f378e 8139864: Improve handling of stack protection zones.
goetz
parents: 33105
diff changeset
    33
#include "runtime/thread.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    34
489c9b5090e2 Initial load
duke
parents:
diff changeset
    35
489c9b5090e2 Initial load
duke
parents:
diff changeset
    36
// Implementation of AbstractAssembler
489c9b5090e2 Initial load
duke
parents:
diff changeset
    37
//
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
// The AbstractAssembler is generating code into a CodeBuffer. To make code generation faster,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
// the assembler keeps a copy of the code buffers boundaries & modifies them when
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
// emitting bytes rather than using the code buffers accessor functions all the time.
2131
98f9cef66a34 6810672: Comment typos
twisti
parents: 1412
diff changeset
    41
// The code buffer is updated via set_code_end(...) after emitting a whole instruction.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    42
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
AbstractAssembler::AbstractAssembler(CodeBuffer* code) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
  if (code == NULL)  return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
  CodeSection* cs = code->insts();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
  cs->clear_mark();   // new assembler kills old mark
14625
b02f361c324e 8003195: AbstractAssembler should not store code pointers but use the CodeSection directly
twisti
parents: 14624
diff changeset
    47
  if (cs->start() == NULL)  {
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 25351
diff changeset
    48
    vm_exit_out_of_memory(0, OOM_MMAP_ERROR, "CodeCache: no room for %s", code->name());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
  }
14625
b02f361c324e 8003195: AbstractAssembler should not store code pointers but use the CodeSection directly
twisti
parents: 14624
diff changeset
    50
  _code_section = cs;
b02f361c324e 8003195: AbstractAssembler should not store code pointers but use the CodeSection directly
twisti
parents: 14624
diff changeset
    51
  _oop_recorder= code->oop_recorder();
b02f361c324e 8003195: AbstractAssembler should not store code pointers but use the CodeSection directly
twisti
parents: 14624
diff changeset
    52
  DEBUG_ONLY( _short_branch_delta = 0; )
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
void AbstractAssembler::set_code_section(CodeSection* cs) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
  assert(cs->outer() == code_section()->outer(), "sanity");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
  assert(cs->is_allocated(), "need to pre-allocate this section");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
  cs->clear_mark();  // new assembly into this section kills old mark
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
  _code_section = cs;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
// Inform CodeBuffer that incoming code and relocation will be for stubs
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
address AbstractAssembler::start_a_stub(int required_space) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
  CodeBuffer*  cb = code();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
  CodeSection* cs = cb->stubs();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
  assert(_code_section == cb->insts(), "not in insts?");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
  if (cs->maybe_expand_to_ensure_remaining(required_space)
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
      && cb->blob() == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
    return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
  set_code_section(cs);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
  return pc();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
// Inform CodeBuffer that incoming code and relocation will be code
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
// Should not be called if start_a_stub() returned NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
void AbstractAssembler::end_a_stub() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
  assert(_code_section == code()->stubs(), "not in stubs?");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
  set_code_section(code()->insts());
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
// Inform CodeBuffer that incoming code and relocation will be for stubs
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
address AbstractAssembler::start_a_const(int required_space, int required_align) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
  CodeBuffer*  cb = code();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
  CodeSection* cs = cb->consts();
14624
8f5ec8cfe196 8003850: add support for constants in stub code
kvn
parents: 13963
diff changeset
    86
  assert(_code_section == cb->insts() || _code_section == cb->stubs(), "not in insts/stubs?");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
  address end = cs->end();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
  int pad = -(intptr_t)end & (required_align-1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
  if (cs->maybe_expand_to_ensure_remaining(pad + required_space)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
    if (cb->blob() == NULL)  return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
    end = cs->end();  // refresh pointer
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
  if (pad > 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
    while (--pad >= 0) { *end++ = 0; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
    cs->set_end(end);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
  set_code_section(cs);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
  return end;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
// Inform CodeBuffer that incoming code and relocation will be code
14624
8f5ec8cfe196 8003850: add support for constants in stub code
kvn
parents: 13963
diff changeset
   102
// in section cs (insts or stubs).
8f5ec8cfe196 8003850: add support for constants in stub code
kvn
parents: 13963
diff changeset
   103
void AbstractAssembler::end_a_const(CodeSection* cs) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
  assert(_code_section == code()->consts(), "not in consts?");
14624
8f5ec8cfe196 8003850: add support for constants in stub code
kvn
parents: 13963
diff changeset
   105
  set_code_section(cs);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
void AbstractAssembler::flush() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
  ICache::invalidate_range(addr_at(0), offset());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
void AbstractAssembler::bind(Label& L) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
  if (L.is_bound()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
    // Assembler can bind a label more than once to the same place.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
    guarantee(L.loc() == locator(), "attempt to redefine label");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
  L.bind_loc(locator());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
  L.patch_instructions((MacroAssembler*)this);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
24425
53764d2358f9 8041415: remove port.{cpp,hpp} files
zgu
parents: 22234
diff changeset
   122
void AbstractAssembler::generate_stack_overflow_check(int frame_size_in_bytes) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
  if (UseStackBanging) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
    // Each code entry causes one stack bang n pages down the stack where n
21528
479228ecf6ac 8026775: nsk/jvmti/RedefineClasses/StressRedefine crashes due to EXCEPTION_ACCESS_VIOLATION
mikael
parents: 17087
diff changeset
   125
    // is configurable by StackShadowPages.  The setting depends on the maximum
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
    // depth of VM call stack or native before going back into java code,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
    // since only java code can raise a stack overflow exception using the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
    // stack banging mechanism.  The VM and native code does not detect stack
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
    // overflow.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
    // The code in JavaCalls::call() checks that there is at least n pages
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
    // available, so all entry code needs to do is bang once for the end of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
    // this shadow zone.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
    // The entry code may need to bang additional pages if the framesize
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
    // is greater than a page.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
    const int page_size = os::vm_page_size();
35201
996db89f378e 8139864: Improve handling of stack protection zones.
goetz
parents: 33105
diff changeset
   137
    int bang_end = (int)JavaThread::stack_shadow_zone_size();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
    // This is how far the previous frame's stack banging extended.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
    const int bang_end_safe = bang_end;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
    if (frame_size_in_bytes > page_size) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
      bang_end += frame_size_in_bytes;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
    int bang_offset = bang_end_safe;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
    while (bang_offset <= bang_end) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
      // Need at least one stack bang at end of shadow zone.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
      bang_stack_with_offset(bang_offset);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
      bang_offset += page_size;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
  } // end (UseStackBanging)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
51633
21154cb84d2a 8209594: guarantee(this->is8bit(imm8)) failed: Short forward jump exceeds 8-bit offset
kvn
parents: 47634
diff changeset
   155
void Label::add_patch_at(CodeBuffer* cb, int branch_loc, const char* file, int line) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
  assert(_loc == -1, "Label is unbound");
41685
7df85c88cdc0 8166562: C2: Suppress relocations in scratch emit.
goetz
parents: 40655
diff changeset
   157
  // Don't add patch locations during scratch emit.
7df85c88cdc0 8166562: C2: Suppress relocations in scratch emit.
goetz
parents: 40655
diff changeset
   158
  if (cb->insts()->scratch_emit()) { return; }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
  if (_patch_index < PatchCacheSize) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
    _patches[_patch_index] = branch_loc;
51633
21154cb84d2a 8209594: guarantee(this->is8bit(imm8)) failed: Short forward jump exceeds 8-bit offset
kvn
parents: 47634
diff changeset
   161
#ifdef ASSERT
21154cb84d2a 8209594: guarantee(this->is8bit(imm8)) failed: Short forward jump exceeds 8-bit offset
kvn
parents: 47634
diff changeset
   162
    _lines[_patch_index] = line;
21154cb84d2a 8209594: guarantee(this->is8bit(imm8)) failed: Short forward jump exceeds 8-bit offset
kvn
parents: 47634
diff changeset
   163
    _files[_patch_index] = file;
21154cb84d2a 8209594: guarantee(this->is8bit(imm8)) failed: Short forward jump exceeds 8-bit offset
kvn
parents: 47634
diff changeset
   164
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
    if (_patch_overflow == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
      _patch_overflow = cb->create_patch_overflow();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
    _patch_overflow->push(branch_loc);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
  ++_patch_index;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
void Label::patch_instructions(MacroAssembler* masm) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
  assert(is_bound(), "Label is bound");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
  CodeBuffer* cb = masm->code();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
  int target_sect = CodeBuffer::locator_sect(loc());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
  address target = cb->locator_address(loc());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
  while (_patch_index > 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
    --_patch_index;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
    int branch_loc;
51633
21154cb84d2a 8209594: guarantee(this->is8bit(imm8)) failed: Short forward jump exceeds 8-bit offset
kvn
parents: 47634
diff changeset
   182
    int line = 0;
21154cb84d2a 8209594: guarantee(this->is8bit(imm8)) failed: Short forward jump exceeds 8-bit offset
kvn
parents: 47634
diff changeset
   183
    const char* file = NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
    if (_patch_index >= PatchCacheSize) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
      branch_loc = _patch_overflow->pop();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
      branch_loc = _patches[_patch_index];
51633
21154cb84d2a 8209594: guarantee(this->is8bit(imm8)) failed: Short forward jump exceeds 8-bit offset
kvn
parents: 47634
diff changeset
   188
#ifdef ASSERT
21154cb84d2a 8209594: guarantee(this->is8bit(imm8)) failed: Short forward jump exceeds 8-bit offset
kvn
parents: 47634
diff changeset
   189
      line = _lines[_patch_index];
21154cb84d2a 8209594: guarantee(this->is8bit(imm8)) failed: Short forward jump exceeds 8-bit offset
kvn
parents: 47634
diff changeset
   190
      file = _files[_patch_index];
21154cb84d2a 8209594: guarantee(this->is8bit(imm8)) failed: Short forward jump exceeds 8-bit offset
kvn
parents: 47634
diff changeset
   191
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
    int branch_sect = CodeBuffer::locator_sect(branch_loc);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
    address branch = cb->locator_address(branch_loc);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
    if (branch_sect == CodeBuffer::SECT_CONSTS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
      // The thing to patch is a constant word.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
      *(address*)branch = target;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
      continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
#ifdef ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
    // Cross-section branches only work if the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
    // intermediate section boundaries are frozen.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
    if (target_sect != branch_sect) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
      for (int n = MIN2(target_sect, branch_sect),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
               nlimit = (target_sect + branch_sect) - n;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
           n < nlimit; n++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
        CodeSection* cs = cb->code_section(n);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
        assert(cs->is_frozen(), "cross-section branch needs stable offsets");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
#endif //ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
    // Push the target offset into the branch instruction.
51633
21154cb84d2a 8209594: guarantee(this->is8bit(imm8)) failed: Short forward jump exceeds 8-bit offset
kvn
parents: 47634
diff changeset
   215
    masm->pd_patch_instruction(branch, target, file, line);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
2148
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   219
struct DelayedConstant {
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   220
  typedef void (*value_fn_t)();
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   221
  BasicType type;
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   222
  intptr_t value;
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   223
  value_fn_t value_fn;
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   224
  // This limit of 20 is generous for initial uses.
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   225
  // The limit needs to be large enough to store the field offsets
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   226
  // into classes which do not have statically fixed layouts.
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   227
  // (Initial use is for method handle object offsets.)
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   228
  // Look for uses of "delayed_value" in the source code
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   229
  // and make sure this number is generous enough to handle all of them.
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   230
  enum { DC_LIMIT = 20 };
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   231
  static DelayedConstant delayed_constants[DC_LIMIT];
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   232
  static DelayedConstant* add(BasicType type, value_fn_t value_fn);
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   233
  bool match(BasicType t, value_fn_t cfn) {
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   234
    return type == t && value_fn == cfn;
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   235
  }
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   236
  static void update_all();
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   237
};
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   238
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   239
DelayedConstant DelayedConstant::delayed_constants[DC_LIMIT];
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   240
// Default C structure initialization rules have the following effect here:
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   241
// = { { (BasicType)0, (intptr_t)NULL }, ... };
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   242
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   243
DelayedConstant* DelayedConstant::add(BasicType type,
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   244
                                      DelayedConstant::value_fn_t cfn) {
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   245
  for (int i = 0; i < DC_LIMIT; i++) {
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   246
    DelayedConstant* dcon = &delayed_constants[i];
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   247
    if (dcon->match(type, cfn))
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   248
      return dcon;
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   249
    if (dcon->value_fn == NULL) {
47634
6a0c42c40cd1 8188220: Remove Atomic::*_ptr() uses and overloads from hotspot
coleenp
parents: 47216
diff changeset
   250
        dcon->value_fn = cfn;
2148
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   251
        dcon->type = type;
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   252
        return dcon;
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   253
    }
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   254
  }
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   255
  // If this assert is hit (in pre-integration testing!) then re-evaluate
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   256
  // the comment on the definition of DC_LIMIT.
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   257
  guarantee(false, "too many delayed constants");
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   258
  return NULL;
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   259
}
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   260
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   261
void DelayedConstant::update_all() {
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   262
  for (int i = 0; i < DC_LIMIT; i++) {
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   263
    DelayedConstant* dcon = &delayed_constants[i];
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   264
    if (dcon->value_fn != NULL && dcon->value == 0) {
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   265
      typedef int     (*int_fn_t)();
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   266
      typedef address (*address_fn_t)();
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   267
      switch (dcon->type) {
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   268
      case T_INT:     dcon->value = (intptr_t) ((int_fn_t)    dcon->value_fn)(); break;
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   269
      case T_ADDRESS: dcon->value = (intptr_t) ((address_fn_t)dcon->value_fn)(); break;
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 41685
diff changeset
   270
      default:        break;
2148
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   271
      }
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   272
    }
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   273
  }
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   274
}
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   275
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11434
diff changeset
   276
RegisterOrConstant AbstractAssembler::delayed_value(int(*value_fn)(), Register tmp, int offset) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11434
diff changeset
   277
  intptr_t val = (intptr_t) (*value_fn)();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11434
diff changeset
   278
  if (val != 0)  return val + offset;
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11434
diff changeset
   279
  return delayed_value_impl(delayed_value_addr(value_fn), tmp, offset);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11434
diff changeset
   280
}
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11434
diff changeset
   281
RegisterOrConstant AbstractAssembler::delayed_value(address(*value_fn)(), Register tmp, int offset) {
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11434
diff changeset
   282
  intptr_t val = (intptr_t) (*value_fn)();
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11434
diff changeset
   283
  if (val != 0)  return val + offset;
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11434
diff changeset
   284
  return delayed_value_impl(delayed_value_addr(value_fn), tmp, offset);
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 11434
diff changeset
   285
}
2148
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   286
intptr_t* AbstractAssembler::delayed_value_addr(int(*value_fn)()) {
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   287
  DelayedConstant* dcon = DelayedConstant::add(T_INT, (DelayedConstant::value_fn_t) value_fn);
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   288
  return &dcon->value;
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   289
}
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   290
intptr_t* AbstractAssembler::delayed_value_addr(address(*value_fn)()) {
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   291
  DelayedConstant* dcon = DelayedConstant::add(T_ADDRESS, (DelayedConstant::value_fn_t) value_fn);
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   292
  return &dcon->value;
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   293
}
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   294
void AbstractAssembler::update_delayed_values() {
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   295
  DelayedConstant::update_all();
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   296
}
09c7f703773b 6812678: macro assembler needs delayed binding of a few constants (for 6655638)
jrose
parents: 2131
diff changeset
   297
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
void AbstractAssembler::block_comment(const char* comment) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
  if (sect() == CodeBuffer::SECT_INSTS) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
    code_section()->outer()->block_comment(offset(), comment);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   303
16368
713209c45a82 8008555: Debugging code in compiled method sometimes leaks memory
roland
parents: 14837
diff changeset
   304
const char* AbstractAssembler::code_string(const char* str) {
713209c45a82 8008555: Debugging code in compiled method sometimes leaks memory
roland
parents: 14837
diff changeset
   305
  if (sect() == CodeBuffer::SECT_INSTS || sect() == CodeBuffer::SECT_STUBS) {
713209c45a82 8008555: Debugging code in compiled method sometimes leaks memory
roland
parents: 14837
diff changeset
   306
    return code_section()->outer()->code_string(str);
713209c45a82 8008555: Debugging code in compiled method sometimes leaks memory
roland
parents: 14837
diff changeset
   307
  }
713209c45a82 8008555: Debugging code in compiled method sometimes leaks memory
roland
parents: 14837
diff changeset
   308
  return NULL;
713209c45a82 8008555: Debugging code in compiled method sometimes leaks memory
roland
parents: 14837
diff changeset
   309
}
713209c45a82 8008555: Debugging code in compiled method sometimes leaks memory
roland
parents: 14837
diff changeset
   310
52462
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   311
bool MacroAssembler::uses_implicit_null_check(void* address) {
823
9a5271881bc0 6716785: implicit null checks not triggering with CompressedOops
coleenp
parents: 1
diff changeset
   312
  // Exception handler checks the nmethod's implicit null checks table
9a5271881bc0 6716785: implicit null checks not triggering with CompressedOops
coleenp
parents: 1
diff changeset
   313
  // only when this method returns false.
52462
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   314
  intptr_t int_address = reinterpret_cast<intptr_t>(address);
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   315
  intptr_t cell_header_size = Universe::heap()->cell_header_size();
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   316
  size_t region_size = os::vm_page_size() + cell_header_size;
2254
f13dda645a4b 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 2148
diff changeset
   317
#ifdef _LP64
f13dda645a4b 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 2148
diff changeset
   318
  if (UseCompressedOops && Universe::narrow_oop_base() != NULL) {
52462
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   319
    // A SEGV can legitimately happen in C2 code at address
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   320
    // (heap_base + offset) if  Matcher::narrow_oop_use_complex_address
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   321
    // is configured to allow narrow oops field loads to be implicitly
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   322
    // null checked
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   323
    intptr_t start = ((intptr_t)Universe::narrow_oop_base()) - cell_header_size;
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   324
    intptr_t end = start + region_size;
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   325
    if (int_address >= start && int_address < end) {
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   326
      return true;
823
9a5271881bc0 6716785: implicit null checks not triggering with CompressedOops
coleenp
parents: 1
diff changeset
   327
    }
9a5271881bc0 6716785: implicit null checks not triggering with CompressedOops
coleenp
parents: 1
diff changeset
   328
  }
2254
f13dda645a4b 6791178: Specialize for zero as the compressed oop vm heap base
kvn
parents: 2148
diff changeset
   329
#endif
52462
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   330
  intptr_t start = -cell_header_size;
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   331
  intptr_t end = start + region_size;
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   332
  return int_address >= start && int_address < end;
823
9a5271881bc0 6716785: implicit null checks not triggering with CompressedOops
coleenp
parents: 1
diff changeset
   333
}
52462
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   334
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   335
bool MacroAssembler::needs_explicit_null_check(intptr_t offset) {
52559
f642ede2eb07 8213795: Force explicit null check on patching placeholder offset
rkennke
parents: 52462
diff changeset
   336
  // The offset -1 is used (hardcoded) in a number of places in C1 and MacroAssembler
f642ede2eb07 8213795: Force explicit null check on patching placeholder offset
rkennke
parents: 52462
diff changeset
   337
  // to indicate an unknown offset. For example, TemplateTable::pop_and_check_object(Register r)
f642ede2eb07 8213795: Force explicit null check on patching placeholder offset
rkennke
parents: 52462
diff changeset
   338
  // calls MacroAssembler::null_check(Register reg, int offset = -1) which gets here
f642ede2eb07 8213795: Force explicit null check on patching placeholder offset
rkennke
parents: 52462
diff changeset
   339
  // with -1. Another example is GraphBuilder::access_field(...) which uses -1 as placeholder
f642ede2eb07 8213795: Force explicit null check on patching placeholder offset
rkennke
parents: 52462
diff changeset
   340
  // for offsets to be patched in later. The -1 there means the offset is not yet known
f642ede2eb07 8213795: Force explicit null check on patching placeholder offset
rkennke
parents: 52462
diff changeset
   341
  // and may lie outside of the zero-trapping page, and thus we need to ensure we're forcing
f642ede2eb07 8213795: Force explicit null check on patching placeholder offset
rkennke
parents: 52462
diff changeset
   342
  // an explicit null check for -1, even if it may otherwise be in the range
f642ede2eb07 8213795: Force explicit null check on patching placeholder offset
rkennke
parents: 52462
diff changeset
   343
  // [-cell_header_size, os::vm_page_size).
f642ede2eb07 8213795: Force explicit null check on patching placeholder offset
rkennke
parents: 52462
diff changeset
   344
  // TODO: Find and replace all relevant uses of -1 with a reasonably named constant.
f642ede2eb07 8213795: Force explicit null check on patching placeholder offset
rkennke
parents: 52462
diff changeset
   345
  if (offset == -1) return true;
f642ede2eb07 8213795: Force explicit null check on patching placeholder offset
rkennke
parents: 52462
diff changeset
   346
52462
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   347
  // Check if offset is outside of [-cell_header_size, os::vm_page_size)
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   348
  return offset < -Universe::heap()->cell_header_size() ||
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   349
         offset >= os::vm_page_size();
4ad404da0088 8213199: GC abstraction for Assembler::needs_explicit_null_check()
rkennke
parents: 51633
diff changeset
   350
}