From 4a61c23ab65a8feb58d1bcdb7cbe8c8bdc36b7d1 Mon Sep 17 00:00:00 2001 From: kobalicek Date: Wed, 5 Jun 2024 00:35:11 +0200 Subject: [PATCH] Enable try mode in RA local's switchToAssignment() This feature has been disabled for a long time so it could be tested properly, but production didn't reveal any issues. When try mode is enabled the RA will try to allocate the reassignment first to avoid possibly having to emit code in a separate block (try mode basically "tries" to emit code before a branch and not as a consequence of it). --- src/asmjit/core/ralocal.cpp | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/asmjit/core/ralocal.cpp b/src/asmjit/core/ralocal.cpp index 95ccd16..194b5a8 100644 --- a/src/asmjit/core/ralocal.cpp +++ b/src/asmjit/core/ralocal.cpp @@ -137,10 +137,6 @@ Error RALocalAllocator::switchToAssignment(PhysToWorkMap* dstPhysToWorkMap, cons dst.initMaps(dstPhysToWorkMap, _tmpWorkToPhysMap); dst.assignWorkIdsFromPhysIds(); - // TODO: Remove this - finally enable this functionality. - if (tryMode) - return kErrorOk; - for (RegGroup group : RegGroupVirtValues{}) { // STEP 1 // ------