[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