From 98e89007a6ed9c9e15a420cf734cf2a663881876 Mon Sep 17 00:00:00 2001 From: Manasha Vetrivelu Date: Fri, 8 Jul 2022 15:42:09 -0400 Subject: [PATCH] Asserts fired due to PROD_WITH_ASSUMES commented out Asserts from Issues: #15440, #15447, #15472, #15482 Signed-off-by: Manasha Vetrivelu --- runtime/compiler/control/CompilationThread.cpp | 4 ++-- runtime/compiler/infra/J9MonitorTable.cpp | 2 +- runtime/compiler/x/codegen/J9TreeEvaluator.cpp | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/runtime/compiler/control/CompilationThread.cpp b/runtime/compiler/control/CompilationThread.cpp index 0e3e15c3a3c..1ecc16e2089 100644 --- a/runtime/compiler/control/CompilationThread.cpp +++ b/runtime/compiler/control/CompilationThread.cpp @@ -4934,7 +4934,7 @@ TR::CompilationInfo::addMethodToBeCompiled(TR::IlGeneratorMethodDetails & detail { if (TR::Options::isAnyVerboseOptionSet()) TR_VerboseLog::writeLineLocked(TR_Vlog_INFO, "Discrepancy for queue size while adding to queue: Before adding numEntries=%d _numQueuedMethods=%d\n", numEntries, _numQueuedMethods); - TR_ASSERT(false, "Discrepancy for queue size while adding to queue"); + //TR_ASSERT(false, "Discrepancy for queue size while adding to queue"); } } cur = getCompilationQueueEntry(); @@ -11599,7 +11599,7 @@ TR::CompilationInfo::replenishInvocationCount(J9Method *method, TR::Compilation } else { - TR_ASSERT(false, "Unexpected value for method->extra = %p (method=%p)\n", TR::CompilationInfo::getJ9MethodExtra(method), method); + //TR_ASSERT(false, "Unexpected value for method->extra = %p (method=%p)\n", TR::CompilationInfo::getJ9MethodExtra(method), method); } } diff --git a/runtime/compiler/infra/J9MonitorTable.cpp b/runtime/compiler/infra/J9MonitorTable.cpp index 8becabd0a3b..208356f88cd 100644 --- a/runtime/compiler/infra/J9MonitorTable.cpp +++ b/runtime/compiler/infra/J9MonitorTable.cpp @@ -252,7 +252,7 @@ J9::MonitorTable::readReleaseClassUnloadMonitor(int32_t compThreadIndex) } else { - TR_ASSERT(false, "comp thread %d does not have classUnloadMonitor", compThreadIndex); + //TR_ASSERT(false, "comp thread %d does not have classUnloadMonitor", compThreadIndex); return -1; // could not release monitor } } diff --git a/runtime/compiler/x/codegen/J9TreeEvaluator.cpp b/runtime/compiler/x/codegen/J9TreeEvaluator.cpp index 5da8cb123cd..db8b09a4c8e 100644 --- a/runtime/compiler/x/codegen/J9TreeEvaluator.cpp +++ b/runtime/compiler/x/codegen/J9TreeEvaluator.cpp @@ -10776,7 +10776,7 @@ void J9::X86::TreeEvaluator::VMwrtbarWithoutStoreEvaluator( { TR::Compilation *comp = cg->comp(); TR_J9VMBase *fej9 = (TR_J9VMBase *)(cg->fe()); - TR_ASSERT(!(comp->getOptions()->realTimeGC()),"Call the real-time barrier"); + //TR_ASSERT(!(comp->getOptions()->realTimeGC()),"Call the real-time barrier"); auto gcMode = TR::Compiler->om.writeBarrierType(); if (node->getOpCode().isWrtBar() && node->skipWrtBar())