[PATCH] D129352: [CodeGen] Limit building time for CodeGenPrepare

[LuoYuanke via Phabricator via llvm-commits]

LuoYuanke added inline comments.


================
Comment at: llvm/lib/CodeGen/CodeGenPrepare.cpp:599
+        if (Changed)
+          FreshBBs.insert(&BB);
+        else if (FuncIterated)
----------------
The sinked BB also changed and may have several sinked BBs. Do we record the all sinked BBs in FreshBBs?


================
Comment at: llvm/lib/CodeGen/CodeGenPrepare.cpp:603
+
+      if (ModifiedDTOnIteration == ModifyDT::ModifyBBDT)
+        DT.reset();
----------------
indent.


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

https://reviews.llvm.org/D129352