<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Aug 21, 2017 at 12:41 PM, Reid Kleckner via Phabricator via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rnk added a comment.<br>
<span class=""><br>
> Don't add any EH edges to the CFG for SEH. In practice, BuildOpts.AddEHEdges is always false in practice from what I can tell, and with SEH every single stmt would have to get an EH edge.<br>
<br>
</span>Since we can't mix C++ EH and SEH, do you think it would be better to reuse the TryTerminatedBlock chain so that we get edges from every call to the __except? That's the approximation of SEH that we actually support in LLVM anyway.<br></blockquote><div><br></div><div>Oh, that's a good idea. It'd mean that a CXXThrow would lead to a __try block, but I suppose that's a feature, not a bug?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">================<br>
Comment at: lib/Analysis/CFG.cpp:2570<br>
+  // All __leaves should go to the code following the __try<br>
+  // (FIXME: or if the __try // has a __finally, to the __finally.)<br>
+  SaveAndRestore<JumpTarget> save_break(SEHLeaveJumpTarget)<wbr>;<br>
----------------<br>
Looks like a `//` got re-wrapped in the comment<br>
<br>
<br>
================<br>
Comment at: test/Sema/warn-unreachable-ms.<wbr>c:23<br>
+  } __except(1) {  // Filter expression should not be marked as unreachable.<br>
+    // Emtpy __except body.<br>
+  }<br>
----------------<br>
typo empty<br>
<br>
<br>
<a href="https://reviews.llvm.org/D36914" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D36914</a><br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div></div>