Add support for untagged LoadField, StoreField, LoadElement, and StoreElement simplfied operators. …

Add support for untagged LoadField, StoreField, LoadElement, and StoreElement simplfied operators. This is needed for untagged accesses of typed array backing stores and generally cleans up.

R=bmeurer@chromium.org
BUG=

Committed: https://code.google.com/p/v8/source/detail?r=22815

[Benedikt Meurer, 2014-08-04 11:53:57]

https://codereview.chromium.org/439223004/diff/1/test/cctest/compiler/test-si...
File test/cctest/compiler/test-simplified-lowering.cc (right):

https://codereview.chromium.org/439223004/diff/1/test/cctest/compiler/test-si...
test/cctest/compiler/test-simplified-lowering.cc:89: FieldAccess access = {true,
JSObject::kMapOffset, Handle<Name>(), Type::Any(),
Can we have an enum here? Free-floating true/false != readable.

[titzer, 2014-08-04 12:10:55]

On 2014/08/04 11:53:57, Benedikt Meurer wrote:
>
https://codereview.chromium.org/439223004/diff/1/test/cctest/compiler/test-si...
> File test/cctest/compiler/test-simplified-lowering.cc (right):
> 
>
https://codereview.chromium.org/439223004/diff/1/test/cctest/compiler/test-si...
> test/cctest/compiler/test-simplified-lowering.cc:89: FieldAccess access =
{true,
> JSObject::kMapOffset, Handle<Name>(), Type::Any(),
> Can we have an enum here? Free-floating true/false != readable.

Done.

[Benedikt Meurer, 2014-08-04 12:18:31]

LGTM with nit.

https://codereview.chromium.org/439223004/diff/20001/src/compiler/simplified-...
File src/compiler/simplified-lowering.cc (right):

https://codereview.chromium.org/439223004/diff/20001/src/compiler/simplified-...
src/compiler/simplified-lowering.cc:196: bool base_is_tagged,
MachineRepresentation representation, Type* type) {
Nit: Use that same enum here as well.

[titzer, 2014-08-04 12:25:44]

On 2014/08/04 12:18:31, Benedikt Meurer wrote:
> LGTM with nit.
> 
>
https://codereview.chromium.org/439223004/diff/20001/src/compiler/simplified-...
> File src/compiler/simplified-lowering.cc (right):
> 
>
https://codereview.chromium.org/439223004/diff/20001/src/compiler/simplified-...
> src/compiler/simplified-lowering.cc:196: bool base_is_tagged,
> MachineRepresentation representation, Type* type) {
> Nit: Use that same enum here as well.


Done.

[titzer, 2014-08-04 12:39:22]

Committed patchset #3 manually as 22815 (presubmit successful).