[SV] Fix heap-use-after-free in recent SVExtractTestCode change.

We iterate over the instances of a module in the instance graph, and
during that iteration, erase some of the uses. This causes a
heap-use-after-free that ASAN detects. A simple fix is to use
llvm::make_early_inc_range, which allows us to safely iterate over a
range while we erase things from the range.

Fixes #4081.

lib/Dialect/SV/Transforms/SVExtractTestCode.cpp

@@ -380,7 +380,7 @@ static void inlineInputOnly(hw::HWModuleOp oldMod,
   hw::InstanceGraphNode *node = instanceGraph.lookup(oldMod);
   OpBuilder b(oldMod);
   bool allInlined = true;
-  for (hw::InstanceRecord *use : node->uses()) {
+  for (hw::InstanceRecord *use : llvm::make_early_inc_range(node->uses())) {
     // If there is no instance, move on.
     hw::HWInstanceLike instLike = use->getInstance();
     if (!instLike) {