PTAL
Also got rid of duplication from RAW_REP list macro, and made smi MSB
usable for
bitset.
https://codereview.chromium.org/176843006/diff/1/src/types.cc
File src/types.cc (right):
https://codereview.chromium.org/176843006/diff/1/src/types.cc#newcode158
src/types.cc:158: return kTaggedPtr & (
On 2014/03/04 11:59:58, Michael Starzinger wrote:
High-level comment: This assumes HeapNumber objects are immutable.
This doesn't
hold for double-fields. Better think that through with Toon.
Good point. As discussed off-line, I think we should re-establish that
invariant (not just for the sake of typing) and give those special
number cells a different instance type.
https://codereview.chromium.org/176843006/diff/1/src/types.cc#newcode274
src/types.cc:274: if (value->IsSmi()) return SignedSmall(region);
On 2014/03/04 11:59:58, Michael Starzinger wrote:
Shouldn't this be tagged?
Good catch.
https://codereview.chromium.org/176843006/diff/1/src/types.h
File src/types.h (right):
https://codereview.chromium.org/176843006/diff/1/src/types.h#newcode103
src/types.h:103: // T->Is(SignedSmall())), and the to check whether a
specific case needs
On 2014/03/04 11:59:58, Michael Starzinger wrote:
nit: Seems to be a "latter" missing.
Done.
https://codereview.chromium.org/176843006/diff/1/src/types.h#newcode172
src/types.h:172: V(Allocated, kReceiver | kFloat | kName)
\
On 2014/03/04 11:59:58, Michael Starzinger wrote:
Why kFloat here?
Good question. I removed this thing entirely, since it should now be
subsumed by proper representation types.
https://codereview.chromium.org/176843006/diff/1/src/types.h#newcode174
src/types.h:174: V(Detectable, kDetectableReceiver | kFloat |
kName) \
On 2014/03/04 11:59:58, Michael Starzinger wrote:
Why kFloat here?
Changed to Number.
https://codereview.chromium.org/176843006/diff/1/src/types.h#newcode175
src/types.h:175: V(NonNumber, kOddball | kAllocated |
kInternal) \
On 2014/03/04 11:59:58, Michael Starzinger wrote:
This way kNonNumber contains kFloat, this seems wrong.
Done.
https://codereview.chromium.org/176843006/diff/1/src/types.h#newcode321
src/types.h:321: enum PrintDim { BOTH_DIMS, SEMANTIC_DIM,
REPRESENTATION_DIM };
On 2014/03/04 11:59:58, Michael Starzinger wrote:
nit: Can we write out "PrintDimension" at least for the name of the
enum type
(not the enum values)?
Done.
https://codereview.chromium.org/176843006/
--
--
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.
