================
Comment at: lib/CodeGen/CGException.cpp:1936
@@ -1935,3 @@
- if (!HaveInsertPoint()) {
- FI.ContBB->eraseFromParent();
- if (FI.ResumeBB)
----------------
rnk wrote:
> Can we check !HaveInsertPoint() && FI.ContBB->hasZeroUses() instead and still
> be clever?
You mean `use_empty()`? I tried that (`if (FI.ContBB.use_empty() &&
(!FI.ResumeBB || FI.ResumeBB.use_empty())`) and that didn't immediately work.
But then I though even if it did, he benefits of being clever seem not worth it
over having more control flow, and I stopped trying immediately and sent this
out as-is :-)
http://reviews.llvm.org/D7918
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
