Add Language Specific Changes for Dynamic BF Scan Ordering

Add language specific changes for Dynamic Breadth First Scan Ordering to allow for proper functionality of Dynamic Breadth First Scan Ordering. Signed-off-by: Jonathan Oommen <jon.oommen@gmail.com>
eclipse · Sep 16, 2020 · 4d754a8 · 4d754a8
1 parent af16e89
commit 4d754a8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/compiler/optimizer/VPHandlers.cpp b/compiler/optimizer/VPHandlers.cpp
@@ -2777,6 +2777,7 @@ TR::Node *constrainIaload(OMR::ValuePropagation *vp, TR::Node *node)
    if (base && base->getClass() && !vp->comp()->getOption(TR_DisableMarkingOfHotFields) &&
       node->getFirstChild()->getOpCode().hasSymbolReference() &&
       node->getFirstChild()->getSymbol()->isCollectedReference() &&
+      !vp->comp()->fej9()->isHotReferenceFieldRequired() &&
       vp->_curBlock->getGlobalNormalizedFrequency(vp->comp()->getFlowGraph()) >= TR::Options::_hotFieldThreshold)
       {
       vp->comp()->fej9()->markHotField(vp->comp(), node->getSymbolReference(), base->getClass(), base->isFixedClass());

diff --git a/gc/base/ObjectModelBase.hpp b/gc/base/ObjectModelBase.hpp
@@ -836,8 +836,7 @@ class GC_ObjectModelBase : public MM_BaseVirtual
 	MMINLINE uint8_t
 	getHotFieldOffset(MM_ForwardedHeader *forwardedHeader)
 	{
-		//return _delegate.getHotFieldOffset(forwardedHeader); //TODO: will be populated once Language specific implementation is ready
-		return U_8_MAX;
+		return _delegate.getHotFieldOffset(forwardedHeader);
 	}
 
 	/**
@@ -850,8 +849,7 @@ class GC_ObjectModelBase : public MM_BaseVirtual
 	MMINLINE uint8_t
 	getHotFieldOffset2(MM_ForwardedHeader *forwardedHeader)
 	{
-		//return _delegate.getHotFieldOffset2(forwardedHeader); //TODO: will be populated once Language specific implementation is ready
-		return U_8_MAX;
+		return _delegate.getHotFieldOffset2(forwardedHeader);
 	}
 
 		/**
@@ -864,8 +862,7 @@ class GC_ObjectModelBase : public MM_BaseVirtual
 	MMINLINE uint8_t
 	getHotFieldOffset3(MM_ForwardedHeader *forwardedHeader)
 	{
-		//return _delegate.getHotFieldOffset3(forwardedHeader); //TODO: will be populated once Language specific implementation is ready
-		return U_8_MAX;
+		return _delegate.getHotFieldOffset3(forwardedHeader);
 	}
 
 	/**