On 2014/10/06 19:24:08, arv wrote:
I fixed the issue with the FunctionState and bailout id.

This does not fix the issue that compiler::AstGraphBuilder::VisitClassLiteral
is
reached if I revert the change to mjsunit.status.

Yes, the assert is there because of this test:
(function TestStrictMode() {
  class C {}

  with ({a: 1}) {
    assertEquals(1, a);
  }

  assertThrows('class C extends function B() { with ({}); return B; }() {}',
               SyntaxError);

})();

TestStrictMode has _two_ dont_optimize_reasons - one is 'with' and another is
class literal. 'with' "wins" due to the way the visitor visits, so the
pipeline.cc check tries to optimize. Let's split this test into a separate file
and disable it until turbo fan has a fix.



https://codereview.chromium.org/631433002/

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to