Revert "[regexp] Clone match info for match indices."

This reverts commit dfd9ceb9.

Reason for revert: Regressions https://chromeperf.appspot.com/group_report?rev=64356 https://crbug.com/1015749

Original change's description:
> [regexp] Clone match info for match indices.
>
> The current behavior for generating match indices simply stashes a
> pointer to the match info and then constructs the indices lazily.
> However, it turns out the match info object used to create the result
> object is the regexp_last_match_info living on native context, and thus
> it can change between the creation of the result object and the generation
> of indices. This cl clones the match info which will be safer.
>
> Bug: v8:9548
> Change-Id: Ia6f26f88fbc22fd09671bf4c579d39a1510b552d
> Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1864585
> Commit-Queue: Joshua Litt <joshualitt@chromium.org>
> Reviewed-by: Jakob Gruber <jgruber@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#64356}

TBR=jgruber@chromium.org,joshualitt@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: v8:9548, chromium:1015749
Change-Id: I9c30b8fb459cf2aa89d920bf061614441250844d
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1870236
Commit-Queue: Jakob Gruber <jgruber@chromium.org>
Reviewed-by: Jakob Gruber <jgruber@chromium.org>
Cr-Commit-Position: refs/heads/master@{#64407}

src/builtins/builtins-regexp-gen.cc

@@ -84,14 +84,6 @@ TNode<JSRegExpResult> RegExpBuiltinsAssembler::AllocateRegExpResult(
                        length, SmiConstant(JSArray::kMaxFastArrayLength)));
   CSA_ASSERT(this, SmiGreaterThan(length, SmiConstant(0)));
 
-  // Clone the match info so we can stash a pointer to it. This is necessary
-  // because it may point to the 'regexp_last_match_info' on native context and
-  // thus could change.
-  // TODO(v8:9870): Now that we clone the last match info, we can make its size
-  // static.
-  TNode<FixedArrayBase> cloned_match_info =
-      CloneFixedArray(match_info, ExtractFixedArrayFlag::kFixedArrays);
-
   // Allocate.
 
   const ElementsKind elements_kind = PACKED_ELEMENTS;
@@ -125,10 +117,10 @@ TNode<JSRegExpResult> RegExpBuiltinsAssembler::AllocateRegExpResult(
   StoreObjectFieldNoWriteBarrier(result, JSRegExpResult::kNamesOffset,
                                  undefined_value);
 
-  // Stash cloned_match_info in order to build JSRegExpResultIndices lazily when
-  // the 'indices' property is accessed.
+  // Stash match_info in order to build JSRegExpResultIndices lazily when the
+  // 'indices' property is accessed.
   StoreObjectField(result, JSRegExpResult::kCachedIndicesOrMatchInfoOffset,
-                   cloned_match_info);
+                   match_info);
 
   // Finish elements initialization.
 

test/mjsunit/harmony/regexp-match-indices.js

@@ -103,12 +103,3 @@
 
   assertEquals(m.indices.groups, {'Z': [4, 5]})
 }
-
-// Match between matches.
-{
-  const re = /a+(?<A>zz)?(?<B>ii)?/;
-  const m = re.exec("xaaazzii");
-  assertTrue(/b+(?<C>cccc)?/.test("llllllbbbbbbcccc"));
-  assertEquals(m.indices, [[1, 8], [4, 6], [6, 8]]);
-  assertEquals(m.indices.groups, {'A': [4, 6], 'B': [6, 8]});
-}