[PATCH] D31964: CodeGen: BlockPlacement: Clear ComputedEdges between functions.

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 4 21:42:31 PDT 2017


echristo added inline comments.


================
Comment at: llvm/trunk/lib/CodeGen/MachineBlockPlacement.cpp:2643
 
+  assert(BlockToChain.empty());
+  assert(ComputedEdges.empty());
----------------
Drive-by nit: Typically we like to have some text in the assert as well to explain why we're checking for such a thing, e.g. assert(BlockToChain.empty() && "Forgot to clear BlockToChain between functions!") or something.


https://reviews.llvm.org/D31964





More information about the llvm-commits mailing list