PTAL at the x64 and ARM back-ends.

https://codereview.chromium.org/11093074/diff/8001/src/full-codegen.cc
File src/full-codegen.cc (right):

https://codereview.chromium.org/11093074/diff/8001/src/full-codegen.cc#newcode607
src/full-codegen.cc:607: __ push(Immediate(scope->GetScopeInfo()));
On 2012/11/21 11:00:21, Michael Starzinger wrote:
The previous comment about push() also applies here.

Done.

https://codereview.chromium.org/11093074/diff/8001/src/scopeinfo.h
File src/scopeinfo.h (right):

https://codereview.chromium.org/11093074/diff/8001/src/scopeinfo.h#newcode142
src/scopeinfo.h:142: return
reinterpret_cast<ModuleDescription*>(FixedArray::cast(description));
On 2012/11/21 07:20:47, Sven Panne wrote:
static_cast should be enough here, I think.

Done.

https://codereview.chromium.org/11093074/

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to