Re: [PATCH] D12412: [CodeGen] Point empty %invoke.cont block to successor

2015-09-04 Thread Reid Kleckner via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL246880: Don't use unreachable as a placeholder, it confuses EmitBlock (authored by rnk). Changed prior to commit: http://reviews.llvm.org/D12412?vs=33347&id=34080#toc Repository: rL LLVM http://revi

Re: [PATCH] D12412: [CodeGen] Point empty %invoke.cont block to successor

2015-09-04 Thread Reid Kleckner via cfe-commits
rnk added a subscriber: rnk. rnk added a comment. I think a better fix would be to insert a non-terminator placeholder that doesn't confuse EmitBlock. http://reviews.llvm.org/D12412 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lis

Re: [PATCH] D12412: [CodeGen] Point empty %invoke.cont block to successor

2015-09-04 Thread Vedant Kumar via cfe-commits
vsk added a comment. Ping http://reviews.llvm.org/D12412 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D12412: [CodeGen] Point empty %invoke.cont block to successor

2015-08-27 Thread Vedant Kumar via cfe-commits
vsk created this revision. vsk added reviewers: ygao, rsmith. vsk added a subscriber: cfe-commits. When partially initializing a global, exception-handling code in the frontend emits a placeholder `%invoke.cont` block with no terminator. This commit adds a suitable BranchInst to the block to pre