[webkit-dev] Why does RELEASE_ASSERT not have an error message?

Saam barati sbarati at apple.com
Tue Feb 21 17:54:40 PST 2017


I thought the main point of moving to SIGTRAP was to preserve register state?

That said, there are probably places where we care more about the message than the registers.

- Saam

> On Feb 21, 2017, at 5:43 PM, Mark Lam <mark.lam at apple.com> wrote:
> 
> Is there a reason why RELEASE_ASSERT (and friends) does not call WTFReportAssertionFailure() to report where the assertion occur?  Is this purely to save memory?  svn blame tells me that it has been this way since the introduction of RELEASE_ASSERT in r140577 many years ago.
> 
> Would anyone object to adding a call to WTFReportAssertionFailure() in RELEASE_ASSERT() like we do for ASSERT()?  One of the upside (side-effect) of adding this call is that it appears to stop the compiler from aggregating all the RELEASE_ASSERTS into a single code location, and this will help with post-mortem crash debugging.
> 
> Any thoughts?
> 
> Mark
> 
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> https://lists.webkit.org/mailman/listinfo/webkit-dev



More information about the webkit-dev mailing list