Commit 61972b11 authored by Shu-yu Guo's avatar Shu-yu Guo Committed by Commit Bot

[regexp] Remove unused flag getter use counters

The compatibility fixes have been standardized.

Bug: chromium:581577
Change-Id: I4ab1df59cbcb4bcbcfe9e3a3c658b2d6b81fe68e
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2633539Reviewed-by: 's avatarJakob Kummerow <jkummerow@chromium.org>
Reviewed-by: 's avatarAdam Klein <adamk@chromium.org>
Commit-Queue: Shu-yu Guo <syg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#72210}
parent 352b9ecb
...@@ -8600,8 +8600,8 @@ class V8_EXPORT Isolate { ...@@ -8600,8 +8600,8 @@ class V8_EXPORT Isolate {
kArrayInstanceProtoModified = 27, kArrayInstanceProtoModified = 27,
kArrayInstanceConstructorModified = 28, kArrayInstanceConstructorModified = 28,
kLegacyFunctionDeclaration = 29, kLegacyFunctionDeclaration = 29,
kRegExpPrototypeSourceGetter = 30, kRegExpPrototypeSourceGetter = 30, // Unused.
kRegExpPrototypeOldFlagGetter = 31, kRegExpPrototypeOldFlagGetter = 31, // Unused.
kDecimalWithLeadingZeroInStrictMode = 32, kDecimalWithLeadingZeroInStrictMode = 32,
kLegacyDateParser = 33, kLegacyDateParser = 33,
kDefineGetterOrSetterWouldThrow = 34, kDefineGetterOrSetterWouldThrow = 34,
......
...@@ -21,7 +21,6 @@ transitioning javascript builtin RegExpPrototypeSourceGetter( ...@@ -21,7 +21,6 @@ transitioning javascript builtin RegExpPrototypeSourceGetter(
const methodName: constexpr string = 'RegExp.prototype.source'; const methodName: constexpr string = 'RegExp.prototype.source';
ThrowTypeError(MessageTemplate::kRegExpNonRegExp, methodName); ThrowTypeError(MessageTemplate::kRegExpNonRegExp, methodName);
} }
IncrementUseCounter(context, SmiConstant(kRegExpPrototypeSourceGetter));
return '(?:)'; return '(?:)';
} }
} }
...@@ -188,8 +188,7 @@ extern enum Flag constexpr 'JSRegExp::Flag' { ...@@ -188,8 +188,7 @@ extern enum Flag constexpr 'JSRegExp::Flag' {
kLinear kLinear
} }
const kRegExpPrototypeOldFlagGetter: constexpr int31 const kNoCounterFlagGetter: constexpr int31 = -1;
generates 'v8::Isolate::kRegExpPrototypeOldFlagGetter';
const kRegExpPrototypeStickyGetter: constexpr int31 const kRegExpPrototypeStickyGetter: constexpr int31
generates 'v8::Isolate::kRegExpPrototypeStickyGetter'; generates 'v8::Isolate::kRegExpPrototypeStickyGetter';
const kRegExpPrototypeUnicodeGetter: constexpr int31 const kRegExpPrototypeUnicodeGetter: constexpr int31
...@@ -223,8 +222,7 @@ macro FlagGetter(implicit context: Context)( ...@@ -223,8 +222,7 @@ macro FlagGetter(implicit context: Context)(
transitioning javascript builtin RegExpPrototypeGlobalGetter( transitioning javascript builtin RegExpPrototypeGlobalGetter(
js-implicit context: NativeContext, receiver: JSAny)(): JSAny { js-implicit context: NativeContext, receiver: JSAny)(): JSAny {
return FlagGetter( return FlagGetter(
receiver, Flag::kGlobal, kRegExpPrototypeOldFlagGetter, receiver, Flag::kGlobal, kNoCounterFlagGetter, 'RegExp.prototype.global');
'RegExp.prototype.global');
} }
// ES6 21.2.5.5. // ES6 21.2.5.5.
...@@ -232,7 +230,7 @@ transitioning javascript builtin RegExpPrototypeGlobalGetter( ...@@ -232,7 +230,7 @@ transitioning javascript builtin RegExpPrototypeGlobalGetter(
transitioning javascript builtin RegExpPrototypeIgnoreCaseGetter( transitioning javascript builtin RegExpPrototypeIgnoreCaseGetter(
js-implicit context: NativeContext, receiver: JSAny)(): JSAny { js-implicit context: NativeContext, receiver: JSAny)(): JSAny {
return FlagGetter( return FlagGetter(
receiver, Flag::kIgnoreCase, kRegExpPrototypeOldFlagGetter, receiver, Flag::kIgnoreCase, kNoCounterFlagGetter,
'RegExp.prototype.ignoreCase'); 'RegExp.prototype.ignoreCase');
} }
...@@ -241,23 +239,21 @@ transitioning javascript builtin RegExpPrototypeIgnoreCaseGetter( ...@@ -241,23 +239,21 @@ transitioning javascript builtin RegExpPrototypeIgnoreCaseGetter(
transitioning javascript builtin RegExpPrototypeMultilineGetter( transitioning javascript builtin RegExpPrototypeMultilineGetter(
js-implicit context: NativeContext, receiver: JSAny)(): JSAny { js-implicit context: NativeContext, receiver: JSAny)(): JSAny {
return FlagGetter( return FlagGetter(
receiver, Flag::kMultiline, kRegExpPrototypeOldFlagGetter, receiver, Flag::kMultiline, kNoCounterFlagGetter,
'RegExp.prototype.multiline'); 'RegExp.prototype.multiline');
} }
transitioning javascript builtin RegExpPrototypeLinearGetter( transitioning javascript builtin RegExpPrototypeLinearGetter(
js-implicit context: NativeContext, receiver: JSAny)(): JSAny { js-implicit context: NativeContext, receiver: JSAny)(): JSAny {
return FlagGetter( return FlagGetter(
receiver, Flag::kLinear, kRegExpPrototypeOldFlagGetter, receiver, Flag::kLinear, kNoCounterFlagGetter, 'RegExp.prototype.linear');
'RegExp.prototype.linear');
} }
// ES #sec-get-regexp.prototype.dotAll // ES #sec-get-regexp.prototype.dotAll
transitioning javascript builtin RegExpPrototypeDotAllGetter( transitioning javascript builtin RegExpPrototypeDotAllGetter(
js-implicit context: NativeContext, receiver: JSAny)(): JSAny { js-implicit context: NativeContext, receiver: JSAny)(): JSAny {
const kNoCounter: constexpr int31 = -1;
return FlagGetter( return FlagGetter(
receiver, Flag::kDotAll, kNoCounter, 'RegExp.prototype.dotAll'); receiver, Flag::kDotAll, kNoCounterFlagGetter, 'RegExp.prototype.dotAll');
} }
// ES6 21.2.5.12. // ES6 21.2.5.12.
...@@ -365,8 +361,6 @@ const kRegExpMatchIsTrueishOnNonJSRegExp: constexpr UseCounterFeature ...@@ -365,8 +361,6 @@ const kRegExpMatchIsTrueishOnNonJSRegExp: constexpr UseCounterFeature
generates 'v8::Isolate::kRegExpMatchIsTrueishOnNonJSRegExp'; generates 'v8::Isolate::kRegExpMatchIsTrueishOnNonJSRegExp';
const kRegExpMatchIsFalseishOnJSRegExp: constexpr UseCounterFeature const kRegExpMatchIsFalseishOnJSRegExp: constexpr UseCounterFeature
generates 'v8::Isolate::kRegExpMatchIsFalseishOnJSRegExp'; generates 'v8::Isolate::kRegExpMatchIsFalseishOnJSRegExp';
const kRegExpPrototypeSourceGetter: constexpr UseCounterFeature
generates 'v8::Isolate::kRegExpPrototypeSourceGetter';
const kRegExpExecCalledOnSlowRegExp: constexpr UseCounterFeature const kRegExpExecCalledOnSlowRegExp: constexpr UseCounterFeature
generates 'v8::Isolate::kRegExpExecCalledOnSlowRegExp'; generates 'v8::Isolate::kRegExpExecCalledOnSlowRegExp';
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment