Addressed comments. Porting to ARM.
https://chromiumcodereview.appspot.com/14146005/diff/33001/src/ia32/stub-cache-ia32.cc
File src/ia32/stub-cache-ia32.cc (right):
https://chromiumcodereview.appspot.com/14146005/diff/33001/src/ia32/stub-cache-ia32.cc#newcode784
src/ia32/stub-cache-ia32.cc:784: __ mov(scratch1, transition);
On 2013/04/24 15:23:00, danno wrote:
Perhaps create a macro-assembler function for this?
Done.
https://chromiumcodereview.appspot.com/14146005/diff/25/src/objects.cc
File src/objects.cc (right):
https://chromiumcodereview.appspot.com/14146005/diff/25/src/objects.cc#newcode7289
src/objects.cc:7289: int current_offset = 0;
On 2013/04/25 13:33:48, danno wrote:
// 0 -> 'verbatim'
Done.
https://chromiumcodereview.appspot.com/14146005/diff/25/src/objects.cc#newcode7296
src/objects.cc:7296: Name* key = GetKey(descriptor);
On 2013/04/25 13:33:48, danno wrote:
// |verbatim| -> |valid|
Done.
https://chromiumcodereview.appspot.com/14146005/diff/25/src/objects.cc#newcode7302
src/objects.cc:7302: ASSERT(other_details.type() != CALLBACKS);
On 2013/04/25 13:33:48, danno wrote:
// Assert not correct?
Done.
https://chromiumcodereview.appspot.com/14146005/diff/25/src/objects.cc#newcode7310
src/objects.cc:7310: result->Set(descriptor, &d, witness);
On 2013/04/25 13:33:48, danno wrote:
Merge this case with the previous one.
Done.
https://chromiumcodereview.appspot.com/14146005/diff/25/src/objects.cc#newcode7339
src/objects.cc:7339: result->CopyFrom(descriptor, other, descriptor,
witness);
On 2013/04/25 13:33:48, danno wrote:
Simplify logic as discussed. Either generate a field or copy, test
should be
unified.
Done.
https://chromiumcodereview.appspot.com/14146005/diff/25/src/objects.cc#newcode7343
src/objects.cc:7343: for (; descriptor < new_size; descriptor++) {
On 2013/04/25 13:33:48, danno wrote:
// |valid| -> new_size
Done.
https://chromiumcodereview.appspot.com/14146005/diff/25/src/property-details.h
File src/property-details.h (right):
https://chromiumcodereview.appspot.com/14146005/diff/25/src/property-details.h#newcode132
src/property-details.h:132: uint8_t DetailsEncoded() {
On 2013/04/25 13:33:48, danno wrote:
Move this encoding to PropertyDetails
Done.
https://chromiumcodereview.appspot.com/14146005/diff/25/src/property-details.h#newcode183
src/property-details.h:183: PropertyDetails
set_representation(Representation representation) {
On 2013/04/25 13:33:48, danno wrote:
CopyForRepresentation
Done.
https://chromiumcodereview.appspot.com/14146005/diff/25/src/runtime.cc
File src/runtime.cc (right):
https://chromiumcodereview.appspot.com/14146005/diff/25/src/runtime.cc#newcode253
src/runtime.cc:253: // most from the beginning. However, since it may go
slow by adding
On 2013/04/25 13:33:48, danno wrote:
Elaborate, please :-) (fast)
Done.
https://chromiumcodereview.appspot.com/14146005/diff/25/src/runtime.cc#newcode267
src/runtime.cc:267: // TODO(verwaest): Support tracking representations
in the boilerplate.
On 2013/04/25 13:33:48, danno wrote:
Remove all this code, insert TODO
Done.
https://chromiumcodereview.appspot.com/14146005/diff/25/src/stub-cache.cc
File src/stub-cache.cc (right):
https://chromiumcodereview.appspot.com/14146005/diff/25/src/stub-cache.cc#newcode939
src/stub-cache.cc:939: Code::StubType type = handlers->at(0)->type();
On 2013/04/25 13:33:48, danno wrote:
Use number_of_valid_maps
Done.
https://chromiumcodereview.appspot.com/14146005/
--
--
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.
