JIT: remove is rare check from finally cloning (#110483)

There are currently few rarely run finallys, so stop checking for them.
Future changes may introduce some, and cloning them can be beneficial
to nearby hot code.

Author: AndyAyersMS

src/coreclr/jit/fgehopt.cpp

@@ -1194,7 +1194,6 @@ PhaseStatus Compiler::fgCloneFinally()
         unsigned    regionBBCount   = 0;
         unsigned    regionStmtCount = 0;
         bool        hasFinallyRet   = false;
-        bool        isAllRare       = true;
         bool        hasSwitch       = false;
 
         for (BasicBlock* const block : Blocks(firstBlock, lastBlock))
@@ -1215,7 +1214,6 @@ PhaseStatus Compiler::fgCloneFinally()
             }
 
             hasFinallyRet = hasFinallyRet || block->KindIs(BBJ_EHFINALLYRET);
-            isAllRare     = isAllRare && block->isRunRarely();
         }
 
         // Skip cloning if the finally has a switch.
@@ -1232,13 +1230,6 @@ PhaseStatus Compiler::fgCloneFinally()
             continue;
         }
 
-        // Skip cloning if the finally is rarely run code.
-        if (isAllRare)
-        {
-            JITDUMP("Finally in EH#%u is run rarely; skipping.\n", XTnum);
-            continue;
-        }
-
         // Empirical studies from CoreCLR and CoreFX show that less
         // that 1% of finally regions have more than 15
         // statements. So, to avoid potentially excessive code growth,