Re: [webkit-changes] [180317] trunk/Source/JavaScriptCore

2015-02-18 Thread Michael Saboff
Let me add more to the first paragraph. On Feb 18, 2015, at 4:12 PM, Michael Saboff msab...@apple.com wrote: Phil is adamant that we should eliminate this from trunk. He believes it is the best solution for avoiding the crash in rdar://problem/17198412 rdar://problem/17198412 CrashTracer:

Re: [webkit-changes] [180317] trunk/Source/JavaScriptCore

2015-02-18 Thread Geoffrey Garen
If there’s something incorrect or incomplete about this change, and/or if it’s gone from TOT, I don’t agree that we should commit it to the branch. Have we committed the change to the branch? Is there still a Radar proposing committing the change to the branch? Geoff On Feb 18, 2015, at 3:52

Re: [webkit-changes] [180317] trunk/Source/JavaScriptCore

2015-02-18 Thread Michael Saboff
Phil is adamant that we should eliminate this from trunk. He believes it is the best solution for avoiding the crash in rdar://problem/17198412 CrashTracer: DFG_CRASH beneath JSC::FTL::LowerDFGToLLVM::compileNode, and thinks it is less risky than the fix for rdar://problem/19187718 Crash in

[webkit-changes] [180317] trunk/Source/JavaScriptCore

2015-02-18 Thread msaboff
Title: [180317] trunk/Source/_javascript_Core Revision 180317 Author msab...@apple.com Date 2015-02-18 15:52:16 -0800 (Wed, 18 Feb 2015) Log Message Rollout r180247 r180249 from trunk https://bugs.webkit.org/show_bug.cgi?id=141773 Reviewed by Filip Pizlo. Theses changes makes sense to