Drop deprecated method for backing store from AccessBuilder.
authormstarzinger@chromium.org <mstarzinger@chromium.org>
Mon, 29 Sep 2014 11:35:01 +0000 (11:35 +0000)
committermstarzinger@chromium.org <mstarzinger@chromium.org>
Mon, 29 Sep 2014 11:35:01 +0000 (11:35 +0000)
R=bmeurer@chromium.org
TEST=cctest/test-simplified-lowering/RunLoadStoreArrayBuffer

Review URL: https://codereview.chromium.org/614703002

git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@24282 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

src/compiler/access-builder.cc
src/compiler/access-builder.h
test/cctest/compiler/test-simplified-lowering.cc

index ac9cfa8..749c04a 100644 (file)
@@ -58,13 +58,6 @@ ElementAccess AccessBuilder::ForFixedArrayElement() {
 
 
 // static
-ElementAccess AccessBuilder::ForBackingStoreElement(MachineType rep) {
-  return {kUntaggedBase, kNonHeapObjectHeaderSize - kHeapObjectTag, Type::Any(),
-          rep};
-}
-
-
-// static
 ElementAccess AccessBuilder::ForTypedArrayElement(ExternalArrayType type,
                                                   bool is_external) {
   BaseTaggedness taggedness = is_external ? kUntaggedBase : kTaggedBase;
index 7d0bda1..72dd023 100644 (file)
@@ -37,9 +37,6 @@ class AccessBuilder FINAL : public AllStatic {
   // Provides access to FixedArray elements.
   static ElementAccess ForFixedArrayElement();
 
-  // TODO(mstarzinger): Raw access only for testing, drop me.
-  static ElementAccess ForBackingStoreElement(MachineType rep);
-
   // Provides access to Fixed{type}TypedArray and External{type}Array elements.
   static ElementAccess ForTypedArrayElement(ExternalArrayType type,
                                             bool is_external);
index e67df9b..bafa2d8 100644 (file)
@@ -235,7 +235,7 @@ TEST(RunLoadStoreArrayBuffer) {
   const int index = 12;
   const int array_length = 2 * index;
   ElementAccess buffer_access =
-      AccessBuilder::ForBackingStoreElement(kMachInt8);
+      AccessBuilder::ForTypedArrayElement(v8::kExternalInt8Array, true);
   Node* backing_store = t.LoadField(
       AccessBuilder::ForJSArrayBufferBackingStore(), t.Parameter(0));
   Node* load =