Use __builtin_trap() for CRASH when building with clang
Created attachment 103789 [details] Patch
As a note, I am not happy with the name ASSERT_NO_RETURN. But, while I think of a better name, I want the bots to run on this.
Second note, this is to make the clang static analyzer be able to reason about ASSERT and CRASH in WebCore!
Comment on attachment 103789 [details] Patch Attachment 103789 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/9351695
Created attachment 103802 [details] Patch
Created attachment 103827 [details] Patch
I have changed the macro to NO_RETURN_DUE_TO_ASSERT (and added NO_RETURN_DUE_TO_CRASH) which I like better as names.
Comment on attachment 103827 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=103827&action=review > Source/WebCore/bindings/objc/WebScriptObject.mm:610 > + > +- (void)dealloc NO_RETURN_DUE_TO_ASSERT; > + Please remove the whitespace here.
Committed r93012: <http://trac.webkit.org/changeset/93012>
*** Bug 26630 has been marked as a duplicate of this bug. ***