[PATCH] D43033: [WinEH] Put funclet bundles on inline asm calls

2018-02-08 Thread Reid Kleckner via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL324689: [WinEH] Put funclet bundles on inline asm calls (authored by rnk, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D43033?vs=133263&id=1

[PATCH] D43033: [WinEH] Put funclet bundles on inline asm calls

2018-02-07 Thread David Majnemer via Phabricator via cfe-commits
majnemer accepted this revision. majnemer added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D43033 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/c

[PATCH] D43033: [WinEH] Put funclet bundles on inline asm calls

2018-02-07 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai added a comment. LGTM, but I'd like @majnemer to take a look, since I'm not very familiar with this code. https://reviews.llvm.org/D43033 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo

[PATCH] D43033: [WinEH] Put funclet bundles on inline asm calls

2018-02-07 Thread Reid Kleckner via Phabricator via cfe-commits
rnk created this revision. rnk added reviewers: smeenai, majnemer. Herald added a subscriber: eraman. Fixes PR36247, which is where WinEHPrepare replaces inline asm in funclets with unreachable. Make getBundlesForFunclet return by value to simplify some call sites. https://reviews.llvm.org/D430