No disagreements. Just commenting. :)<br><br><div class="gmail_quote"><div dir="ltr">On Wed, Mar 9, 2016, 4:05 PM Mark Lacey <<a href="mailto:mark.lacey@apple.com">mark.lacey@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rudkx added a comment.<br>
<br>
Thanks for taking a look!<br>
<br>
I agree that it's a bit odd that both do the same for MSVC, but it seems like the right fix for that issue is to change LLVM_BUILTIN_TRAP to fail differently than using __debugbreak (either RaiseException which appears to be deliberately avoided, or inline asm with ud2 perhaps?).<br>
<br>
<br>
<a href="http://reviews.llvm.org/D18002" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18002</a><br>
<br>
<br>
<br>
</blockquote></div>