https://codereview.chromium.org/106453003/diff/100001/src/arm/code-stubs-arm.cc
File src/arm/code-stubs-arm.cc (right):
https://codereview.chromium.org/106453003/diff/100001/src/arm/code-stubs-arm.cc#newcode339
src/arm/code-stubs-arm.cc:339: }
Probably by accident.
https://codereview.chromium.org/106453003/diff/100001/src/hydrogen.cc
File src/hydrogen.cc (right):
https://codereview.chromium.org/106453003/diff/100001/src/hydrogen.cc#newcode1688
src/hydrogen.cc:1688: HAllocate* HGraphBuilder::BuildAllocate(
Yep, that's the plan.
https://codereview.chromium.org/106453003/diff/100001/src/hydrogen.cc#newcode1718
src/hydrogen.cc:1718: // supported string length, we bailout to the
runtime. This is done implicitly
:-)
https://codereview.chromium.org/106453003/diff/100001/src/hydrogen.cc#newcode1746
src/hydrogen.cc:1746: // one-byte or two-byte and set the appropriate
map.
We don't need to. Discussed offline.
https://codereview.chromium.org/106453003/diff/100001/src/hydrogen.cc#newcode1790
src/hydrogen.cc:1790: AddStoreMapConstantNoWriteBarrier(result, map);
Not in case of pretenuring, unfortunately.
https://codereview.chromium.org/106453003/diff/100001/src/hydrogen.cc#newcode1795
src/hydrogen.cc:1795: Handle<Map> map =
isolate()->factory()->cons_string_map();
Not in case of pretenuring, unfortunately.
https://codereview.chromium.org/106453003/diff/100001/src/hydrogen.cc#newcode1950
src/hydrogen.cc:1950: AddStoreMapNoWriteBarrier(result, map);
I don't know.
https://codereview.chromium.org/106453003/
--
--
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.
