[PATCH] D108657: [CodeExtractor] Ensuring ordering of exitStub creation

[Jessica Paquette via Phabricator via llvm-commits]

paquette added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/CodeExtractor.cpp:1292
     for (unsigned i = 0, e = TI->getNumSuccessors(); i != e; ++i)
       if (!Blocks.count(TI->getSuccessor(i))) {
         BasicBlock *OldTarget = TI->getSuccessor(i);
----------------
Reduce indentation?

```
 if (Blocks.count(TI->getSuccessor(i)))
   continue;
...
```


================
Comment at: llvm/lib/Transforms/Utils/CodeExtractor.cpp:1657
+    for (unsigned i = 0, e = TI->getNumSuccessors(); i != e; ++i)
+      if (!Blocks.count(TI->getSuccessor(i))) {
+        BasicBlock *OldTarget = TI->getSuccessor(i);
----------------
reduce indentation here too?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108657/new/

https://reviews.llvm.org/D108657