Oops. I think I looked at the wrong patch set or something. Indeed it supports multiple returns.
Vyacheslav Egorov On Jun 4, 2013 7:59 AM, <[email protected]> wrote: > > https://codereview.chromium.**org/14253015/diff/29001/src/** > ia32/lithium-codegen-ia32.cc<https://codereview.chromium.org/14253015/diff/29001/src/ia32/lithium-codegen-ia32.cc> > File src/ia32/lithium-codegen-ia32.**cc (right): > > https://codereview.chromium.**org/14253015/diff/29001/src/** > ia32/lithium-codegen-ia32.cc#**newcode2782<https://codereview.chromium.org/14253015/diff/29001/src/ia32/lithium-codegen-ia32.cc#newcode2782> > src/ia32/lithium-codegen-ia32.**cc:2782: void LCodeGen::DoReturn(LReturn* > instr) { > On 2013/06/03 13:51:03, Vyacheslav Egorov (Google) wrote: > >> optimized code can have arbitrary many HReturns unlike non-optimized >> > code which > >> branches to the single return sequence from all return points. >> > > Profiler part is designed to allow multiple returns, did I miss some > points in the code generator where returns should be reported? > > https://codereview.chromium.**org/14253015/<https://codereview.chromium.org/14253015/> > -- -- 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/groups/opt_out.
