Review comments

src/coreclr/jit/importer.cpp

@@ -11471,16 +11471,14 @@ void Compiler::impImportBlock(BasicBlock* block)
         // This will re-import all the successors of block (as well as each of their predecessors)
         impReimportSpillClique(block);
 
-        // For blocks that haven't been imported yet, we still need to mark them as pending import.
-        // Filter successor from BBJ_EHFILTERRET have already been handled, above.
-        if (!block->KindIs(BBJ_EHFILTERRET))
+        // We don't expect to see BBJ_EHFILTERRET here.
+        assert(!block->KindIs(BBJ_EHFILTERRET));
+
+        for (BasicBlock* const succ : block->Succs())
         {
-            for (BasicBlock* const succ : block->Succs())
+            if ((succ->bbFlags & BBF_IMPORTED) == 0)
             {
-                if ((succ->bbFlags & BBF_IMPORTED) == 0)
-                {
-                    impImportBlockPending(succ);
-                }
+                impImportBlockPending(succ);
             }
         }
     }
@@ -11489,7 +11487,8 @@ void Compiler::impImportBlock(BasicBlock* block)
         // otherwise just import the successors of block
 
         // Does this block jump to any other blocks?
-        // Filter successor from BBJ_EHFILTERRET have already been handled, above.
+        // Filter successor from BBJ_EHFILTERRET have already been handled above in the call
+        // to impVerifyEHBlock().
         if (!block->KindIs(BBJ_EHFILTERRET))
         {
             for (BasicBlock* const succ : block->Succs())