Re: [webkit-dev] Change WTFCrash to not trash the crash site register state.

2016-02-09 Thread Geoffrey Garen
I like this change. Perhaps all ports can adopt this behavior. Geoff > On Feb 8, 2016, at 11:55 AM, Mark Lam wrote: > > Hi WebKit folks, > > For non-debug OS(DARWIN) builds, I would like to change WTFCrash()’s > implementation into an inlined function that has a single inlined asm > statem

Re: [webkit-dev] Change WTFCrash to not trash the crash site register state.

2016-02-08 Thread Filip Pizlo
Makes sense to me. -Filip > On Feb 8, 2016, at 12:33 PM, Mark Lam wrote: > > A store to 0xbbadbeef will still require the use of a register (at least on > ARM). The breakpoint instruction uses no registers (hence, we don’t have to > choose which register to sacrifice). We can still identif

Re: [webkit-dev] Change WTFCrash to not trash the crash site register state.

2016-02-08 Thread Mark Lam
A store to 0xbbadbeef will still require the use of a register (at least on ARM). The breakpoint instruction uses no registers (hence, we don’t have to choose which register to sacrifice). We can still identify the crash as an assertion by looking fro the EXC_BREAKPOINT instead of the 0xbbadbe

Re: [webkit-dev] Change WTFCrash to not trash the crash site register state.

2016-02-08 Thread Filip Pizlo
I like this idea. I’ve wanted this for a while. Can you explain why your approach doesn’t inline a store to 0xbbadbeef, so that this aspect of the current behavior is preserved? -Filip > On Feb 8, 2016, at 11:55 AM, Mark Lam wrote: > > Hi WebKit folks, > > For non-debug OS(DARWIN) builds,

[webkit-dev] Change WTFCrash to not trash the crash site register state.

2016-02-08 Thread Mark Lam
Hi WebKit folks, For non-debug OS(DARWIN) builds, I would like to change WTFCrash()’s implementation into an inlined function that has a single inlined asm statement that issues a breakpoint trap. The intent is to crash directly in the caller’s frame and preserve the register values at the tim