[master] Update dependencies from dotnet/runtime-assets dotnet/icu (#48698)

[master] Update dependencies from dotnet/runtime-assets dotnet/icu

Tweak package build for dotnet-pgo so that it builds in our official builds (#48724)

Preserve the System.RuntimeType constructor in mono ILLink (#48710)

* Preserve the System.RuntimeType constructor in mono ILLink

* Update src/mono/System.Private.CoreLib/src/ILLink/ILLink.Descriptors.xml

* Update src/mono/System.Private.CoreLib/src/ILLink/ILLink.Descriptors.xml

Co-authored-by: Larry Ewing <lewing@microsoft.com>
Co-authored-by: Larry Ewing <lewing@microsoft.com>

Make Exception.StackTrace resilient to missing attribute assemblies (#48535)

A customer has reported a problem when they could not get stack trace
from an exception that they needed to log, because one of the methods on
the stack or its defining type had an attribute from an assembly that
was missing.
This change fixes that by ignoring exceptions in attempts to get
StackTraceHiddenAttribute, preferring to err on the side of getting a
stack frame that was marked as hidden in the stack trace rather than
not getting any stack trace at all.

Replace non-generic WeakReference usage in several libraries (#48618)

* Replace non-generic WeakReference usage in System.Private.Runtime.InteropServices.JavaScript

* Replace non-generic WeakReference usage in System.Private.Xml

* Replace non-generic WeakReference usage in System.Diagnostics.TraceSource

* Address PR feedback

Enhance coreclr processing of event pipe configuration to be able to write the pid of the current process into the nettrace file (#48537)

- Also fix the behavior of get_next_config_value_as_utf8_string to work correctly on coreclr, it was type-punning between a buffer returned by ep_rt_byte_array_alloc and ep_rt_utf8_string_free, which isn't safe in the coreclr implementation

Fixup richnav triggers (#48714)

Turn off LKG calculation (#48708)

R2RTest cleanup for Serp compile (#48528)

* Simplify compilation logic now a single mega composite image is functional
* Remove composite scenario switch and support just composite or individual assemblies
* Use the same list of allowed Dlls in the serp/bin directory that Serp deployment uses

Fix warning for Rfc2898OneShotTests.cs (#48688)

Spill 'RET_EXPR' in fat call arguments. (#48690)

Hot reload for xplat change broke DAC (#48672)

On Linux/MacOS only enabled ENC for the debug/ee and vm directories and that
through some the EE's data structure's vtables between the runtime and DAC.

Updating snaps (#48668)

[browser][crypto] Fix Quota Exceeded error (#48692)

* Add batched implementation to fix error with quota exceeded limit

* Add and/or update tests for exceeding the quota limit of browser set at 65536.

* Update to test quota exceeded

* Code to use views and write directly to heap

Enable async-over-sync Task in Dns to be canceled (#48684)

While we can't currently cancel the synchronous call, we can at least cancel the Task prior to its being executed.

Update dependencies from https://github.com/dotnet/xharness build 20210224.1 (#48693)

[master] Update dependencies from dotnet/xharness

Fix MailAddress.TryCreate behavior (#47574)

* change tests

* fix TryParse behavior

* fix spacing

* review improvements, change tests

* update throwing exception

* Update src/libraries/System.Net.Mail/tests/Unit/MailAddressTests/MailAddressParsingTest.cs

* Update src/libraries/System.Net.Mail/tests/Functional/MailAddressTest.cs

Co-authored-by: Marie Píchová <11718369+ManickaP@users.noreply.github.com>

Introduce FileStreamStrategy as a first step of FileStream rewrite (#47128)

Co-authored-by: Stephen Toub <stoub@microsoft.com>
Co-authored-by: David Cantú <dacantu@microsoft.com>

Forward Span-based MemoryExtensions overloads to ReadOnlySpan overloads (#48669)

Import iOS from MacCatalyst in libraries OSGroups.json (#48652)

This allows us to reuse the existing iOS build configs since we don't have MacCatalyst-specific logic yet.

Support SPMI collection of tiered compilation runs, fix utf8 issues (#48665)

Add new flag `--tiered_compilation` to enable collection of runs with tiered
compilation enabled (by not setting `COMPlus_TieredCompilation=0`).

Note `COMPlus_TieredCompilation` has no impact on prejitting or on replay.

Also be more tolerant of utf-8 encoding issues, in case we see asserts in
methods or classes with funny names.

Threadpool autorelease (#47592)

Fix indent (#48676)

* Fix yml indenting

* Fix one other indent issue

* Fix comment error

Add arm64 mono (#48671)

* Add Mono interpreter ARM64 perf jobs

* WIP

* Add nightly Arm64 interpreter runs

* Remove test category

Add progressbar to SuperPMI.py script (#48649)

Update workflow doc to include a rough disk size minimum requirement (#48657)

* Update workflow doc to include a rough disk size minimum requirement

Revert "Fix installer build on Alpine (#48505)" (#48651)

This reverts commit c9fe433ebb3e5a489083fda8582861926596a41e.

Throw an ObjectDisposedException instead of InvalidOperationException after Process object has been disposed (#46593)

fixes #45149

Make it clear the subset flag can be omitted (#47691)

* Make it clear the subset flag can be omitted

* Just mention that the subset flag can be implicit

Update dependencies from https://github.com/dotnet/icu build 20210222.1 (#48644)

[master] Update dependencies from dotnet/icu

Enable char.Is{High}Surrogate(string, int) to be trimmed in default Blazor wasm app (#48621)

And simpler/faster code.

Compile native AppleCryptoNative_SecCopyErrorMessageString for ios/tvos

It's referenced from a lot of managed placed

Fix C# warning in Comparer_get_Default (#48643)

implement test for Long Paths support in ProcessModule.FileName (#46685)

Add a Mono ARM64 runtime pack & test build (#48532)

issue-44801 EnsureCapacity Apis For List Stack Queue (#47149)

* issue-44801 Initial commit: public List.EnsureCapacity.

* Create Stack.EnsureCapacity().

* issue-44801 Fix comment for Stack.

* issue-44801 Fix comment: Move temp MaxArrayLength into near usage scope for Stack.

* issue-44801 Fix comment for List.

* issue-44801 Create EnsureCapacity for Queue.

* issue-44801 Create tests for List.EnsureCapacity.

* issue-44801 Create tests for Stack.EnsureCapacity.

* issue-44801 Create tests for Queue.EnsureCapacity.

* issue-44801 Update version if updating internal buffer.

* issue-44801 Update test cases to verify invalidating enumeration (List, Stack, Queue).

* issue-44801 Improve code change for List:

1. Avoid incrementing version number twice in one Insert (or Add) method call;
2. Avoid more capacity check for Insert (or Add) method.

* issue-44801 Fix comments: consider integer overflow; refactor methods.

* Update src/libraries/System.Private.CoreLib/src/System/Collections/Generic/List.cs

Fix comment: update xml doc.

Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>
* ensure same algorithm is used for all resize operations

* revert array length check in List.EnsureCapacityCore

* remove GrowFactor constant

* skip large capacity tests on mono.

* Fix overflow handling when capacity < MaxArrayLength; add clarifying comments

Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>

[master] Update dependencies from dotnet/runtime dotnet/xharness (#48585)

[master] Update dependencies from dotnet/runtime dotnet/xharness

 - Merge branch 'master' into darc-master-d9d4c102-0da3-4e51-b50a-daa5955aadce

Fix ArgIterator::GetNextOffset for macOS arm64 (#48632)

* Fix ArgIterator::GetNextOffset for macOS arm64

The ArgIterator::GetNextOffset method was not aligning offsets it returns
for reference types and some others. That was causing coreclr test failures
with GCStress mode 3.

This change fixes it by properly aligning the returned offsets.

* Fix indentation

Fix installer build on Alpine (#48505)

* Fix installer build on Alpine

* Update src/tests/build.sh

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>

Build unsupported version of System.Security.Cryptography.OpenSsl for mobile targets (#48595)

Co-authored-by: Adeel Mujahid <3840695+am11@users.noreply.github.com>
Co-authored-by: Jeremy Barton <jbarton@microsoft.com>

CA1200 -> Info (#48612)

Co-authored-by: carlossanlop <carlossanlop@users.noreply.github.com>

Remove 'domain' arguments from the reflection object creation functions. (#48586)

* Remove 'domain' arguments from the reflection object creation functions.

* Remove some more domain usage.

fix an old test (#48627)

Refactor code that considers guarded devirtualization (#48611)

Merge two similar hunks of code.

Also, unblock guarded devirtualization when prejitting, as we may see class
guesses via static PGO.

Special-case Enumerable.SequenceEqual for byte[] (#48287)

Add ActivitySource.CreateActivity methods (#48374)

* Add ActivitySource.CreateActivity methods

* Address the feedback

* Add IdFormat parameter to ActivitySource.Create and fix the whole IdFormat logic

* Address @noahfalk feedback

Set nuspec packaging properties correctly (#48625)

Define PackageProjectUrl which is the public facing property to specify
a project url in a nuspec. Arcade defaults PackageProjectUrl to
RepositoryUrl if it isn't set. RepositoryUrl was incorrectly set to
git://github.com/dotnet/runtime instead of
https://github.com/dotnet/runtime which cause PackageProjectUrl to use
that incorrect value as well which ultimately results in nuget.org
failing package validation and refusing to add the package.

Also changing ProjectUrl to the previous https://dot.net value which is
the expected url to the dotnet project. In addition to that, specifying
Owners which aren't set anymore in 6.0.

Adding the ProjectUrl property as well as the
Microsoft.DotNet.Build.Tasks.Packaging package depends on it. That line
can be removed after the fix to not override its value unconditionally
is merged and consumed.

Remove unnecessary msbuild-sdk from global.json (#48599)

* Remove unnecessary msbuild-sdk from global.json

This line was meant to delineate between sdks that were auto-updated by different PRs. But we have since switched to no longer auto-update these SDKs, so there is no reason for this line anymore.

* Reorder so hard-coded entries are in the middle.

Use SHA256 one-shot for CrlCache

Co-authored-by: Jeremy Barton <jbarton@microsoft.com>

Add static PBKDF2 one-shot methods

This avoids the disposable object creation for Rfc2989DeriveBytes and the
state tracking.

Since the one-shots just call into platform routines (when available) it
should allow for higher iteration counts for the same clock time.

EmptyAiaResponseIsIgnored should use incomplete chain.

Since the certificate names were the same, different tests were
colliding with caches since they had the same certificate name.

In addition to the simple fix, this changes the test to build a partial
chain using two intermediates. The leaf's issuing intermediate is
added to the ExtraStore and the second intermediate is expected
to be fetched by AIA (and fail with an empty response).

This is to help coerce chain builders treat it as a partial chain,
and can actually make some progress, as opposed to the leaf being
treated as an invalid root.

Make sure pal_icushim_static.c is linked in even if no symbols are used from it, so the EMSCRIPTEN_KEEPALIVE functions in it are actually kept. (#48550)

Support PolynomialMultiplyWideningLower and PolynomialMultiplyWideningUpper (#48525)

Minor msbuild code cleanup in installer partition (#48603)

* Minor code clean-up in installer partition

* Move prepare-artifacts.proj into installer root

Package dotnet-pgo as a dotnet tool (#48542)

- The package is marked as non-shipping as we're not ready to make it part of the official product at this time, but this should make it easier to consume in downstream repos.

Transition JIT memory management from domains to memory managers (#48483)

* [jit] Add a MonoJitMemoryManager structure, which is the same as the MonoJitDomainInfo structure, but owned by a MonoMemoryManager instead of a domain.

* Move static_rgctx_trampoline_hash to MonoJitMemoryManager.

* Move jump_target_hash to MonoJitMemoryManager.

* Move jump_trampoline_hash to MonoJitMemoryManager.

* Move jit_trampoline_hash to MonoJitMemoryManager.

* Move delegate_trampoline_hash to MonoJitMemoryManager.

* Move method_code_hash to MonoJitMemoryManager.

* Move more fields to MonoJitMemoryManager.

* Move llvm data to MonoJitMemoryManager.

* Move arch_seq_points field to MonoJitMemoryManager.

* Remove unused fields from MonoJitDomainInfo.

* Use the default alc memory manager only for now.

* Remove MonoJitDomainInfo which is now unused.

Read basic properties of a X509 certificate using Android-built-in Crypto APIs (#48425)

Remove hardcoded PackageVersion in Platforms pkg (#48596)

Mapping the package to `$(ProductVersion)` would be wrong in servicing cases. Hence just removing the hardcoded value entirely.

Fix ThreadPool events validation test (#48588)

* Fix ThreadPool events validation test

* space

Remove unaligned offsets for mapped layout (#42464)

* Remove unaligned offsets for mapped layout

* Assert on windows only

- Unix will use the mapped layout with an offset for bundled assemblies
- Windows still should't create a mapped layout for bundles

[master] Update dependencies from mono/linker (#48597)

Microsoft.NET.ILLink.Tasks
 From Version 6.0.100-preview.2.21118.4 -> To Version 6.0.100-preview.2.21122.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>

Delete publish-final.proj and clean-up installer.tasks unused files (#48598)

* Delete publish-final.proj and delete unused installer.tasks files

* Avoid extra restore phase for tasks

The repo local tasks don't participate in the repo restore anymore,
hence avoiding the extra restore phase which should improve reliability
and build times.

Delete updateversions.proj (#48593)

dead code

Fix doc comment typo in JsonSerializerOptions.cs (#48590)

The following sentence in the doc comments for the `IgnoreReadOnlyFields` member:

> A property is read-only if it isn't marked with the <c>readonly</c> keyword.

Should read:

> A field is read-only if it is marked with the <c>readonly</c> keyword.

Fix another doc comment typo in JsonSerializerOptions.cs (#48591)

* Fix another doc comment typo in JsonSerializerOptions.cs

The following sentence in the doc comments for the `IncludeFields` member:

> Determines whether fields are handled serialization and deserialization.

Should read:

> Determines whether fields are handled **during** serialization and deserialization.

* Update src/libraries/System.Text.Json/src/System/Text/Json/Serialization/JsonSerializerOptions.cs

Co-authored-by: Eirik Tsarpalis <eirik.tsarpalis@gmail.com>

Add LINQ APIs for Index and Range (#28776) (#48559)

* Implement dotnet/runtime#28776: Implement LINQ APIs for index and range.

* Implement dotnet#28776: Implement LINQ APIs for index and range.

* Implement dotnet#28776: LINQ APIs for index and range.

* Implement dotnet#28776: LINQ APIs for index and range.

* Implement dotnet#28776: LINQ APIs for index and range.

* Implement dotnet#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Code review update.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Code review update.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Code review update.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Update ElementAt, keep the original behavior.

* Implement dotnet#28776: LINQ APIs for index and range. Update ElementAt, keep the original behavior.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Update ElementAt, keep the original behavior.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Add unit tests for ElementAt, ElementAtOrDefault.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Update unit tests.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Update unit tests.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Update unit tests.

* Implement dotnet/runtime#28776: LINQ APIs for index and range. Update for merge.

* Implement #28776: LINQ APIs for index and range. Update unit tests.

[interp] Inline more AggressiveInlining methods (#48513)

* [interp] Inline more AggressiveInlining methods

By default, the interpreter stops inlining when encountering a call that it cannot inline or a throw. Inline the method anyway if the aggressive inlining attribute is present.

* [interp] Fix some long conditional branch opcodes

Delete unused installer properties (#48569)

* Delete unused msbuild properties, targets and files

Improve S.DS.P tests, fix async API, and improve error messages (#47703)

Simplify connection usability logic  (#48446)

* simplify connection usability logic

Co-authored-by: Geoffrey Kizer <geoffrek@windows.microsoft.com>

JIT: remove JitGuardedDevirtualizationGuessBestClass (#48574)

And slightly rearrange the two bits of code that consider guarded
devirualization with the goal of eventually unifying them.

Fixes #48564.

Add disabling test policy to the docs (#48575)

* add disabling test policy

* Update docs/infra/buildtriage.md

Co-authored-by: Dan Moseley <danmose@microsoft.com>

Fix #28035 Refactor DLL imports interop classes (#48556)

* refactor interop classes

* move interop files to subdirectory

Enable CG2 determinism tests on all architectures (#48526)

I have noticed that these two tests are still conditionally
executed on x64 only. I don't see why they shouldn't work on
other architectures.

Thanks

Tomas

update branding to preview3 (#48530)

Reuse SDK target for latest commit hash (#48561)

* Reuse SDK target for latest commit hash

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>
* Further clean-up corehost.proj

Co-authored-by: Viktor Hofer <viktor.hofer@microsoft.com>

Add assembly name to SuperPMI collection (#48534)

* Add assembly name to SuperPMI collection

For each class, add JIT/EE calls to getAssemblyName so the SuperPMI
collections contain this information. This can be useful to try and track
down where a particular function in a large method context hive came
from.

Only do this when `COMPlus_EnableExtraSuperPmiQueries=1` is set.

* Formatting

Enable Hot Reload API on Linux (#48497)

Enable Hot Reload API on Linux

Only define EnC_SUPPORTED for vm/ee on xplat. Doesn't enable the ENC ICorDebug API.

Enable hotreload api tests on Linux/MacOS. API not implemented on arm/arm64

Remove all the ENC_DELTA_HACK code. Remove more ENC_DELTA_HACK code from ilasm

[mono] Remove MonoRuntimeUnhandledExceptionPolicy (#48555)

policy was always set to MONO_UNHANDLED_POLICY_CURRENT.  The
legacy (pre-.NET 2.0) policy was not used.

JIT: workaround for unreliable change detection in fgReorderBlocks (#48516)

This method costs trees, which can in turn modify the IR by swapping operands.
As a result the bool value returned doesn't properly reflect whether any
changes happened.

This impacts proper reporting phase status by `optOptimizeLayout'. Since phase
status just gates post-phase dumps and checks, we'll just claim this phase
always modifies IR.

Add similar workaround for `optInvertLoops`.

Fixes #48494.
Fixes #48495.

Fix JIT rolling build (#48552)

Update hardcoded rids for targeting/runtime pks (#48459)

* Update hardcoded rids for targeting/runtime pks

Format (#48548)

Building singlefilehost with clr partition. (#48254)

* building single file host with clr partition

* make coreclr_static a STATIC lib again

[mono] Implement public hot reload API (#48380)

* [mono] Implement public hot reload API

Also add ApplyUpdateSdb that takes byte[] arguments for use with
mono/debugger-libs as its awkward to call a ROS<byte> method from the debugger.

Contributes to https://github.com/dotnet/runtime/issues/45689

* Update src/mono/System.Private.CoreLib/src/System/Reflection/Metadata/AssemblyExtensions.cs

Co-authored-by: Marek Safar <marek.safar@gmail.com>
* Don't allow changes to System.Private.CoreLib

* formatting

* more formatting

* Add a TODO to remove ApplyUpdateSdb

* update DeltaHelper

* don't use ApplyUpdateSdb in DeltaHelper

* throw InvalidOpertionException for corlib changes with hot reload disabled

* Run the test on mono on all platoforms; CoreCLR windows only

* Use resource string

* Address review feedback

Co-authored-by: Marek Safar <marek.safar@gmail.com>

Temporarily work around publishing issues with disk space (#48521)

Wasm download fix (#48514)

* Don't use container for WASM

* Remove dashes

* Remove comments for testing

Disable TestOnExecuteCustomCommand() for WSC (#48501)

Remove StringBuilderCache from Version.ToString (#48511)

Rather than using a StringBuilder in both ToString and TryFormat, we can just write ToString in terms of TryFormat and use stack space for the small string being generated.  As part of this, I removed the extra code optimizing ToString(0) and ToString(1), as I can't find any real code anywhere that does that and it's not worth the extra lines.

Fix EqualityComparer<T>.Default for nullable (#48161)

No const arg to temp (#48308)

* Starting to add IsInvariant helper

* Finished IsInvariant helper. Starting testing

* Removed IsInvariant from GenTree to avoid asserts

* Formatting

* Fixed missing not operator

* Changed AsOp() call to 'this' to avoid failing assertions

* Formatting

* Update src/coreclr/jit/morph.cpp

Co-authored-by: Sergey Andreenko <seandree@microsoft.com>
* Update src/coreclr/jit/compiler.h

Co-authored-by: Sergey Andreenko <seandree@microsoft.com>
* Apply suggestions from code review

Co-authored-by: Sergey Andreenko <seandree@microsoft.com>
* Change arguments to const to avoid cast

* Formatting

* delete `IsInvariant` from Compiler.

Co-authored-by: Sergey Andreenko <seandree@microsoft.com>

Don't build Quic for platforms where it's not supported (#48480)

Annotate framework assemblies as trimmable (#48428)

* Annotate framework assemblies as trimmable

This will opt them into trimming with the behavior described at https://github.com/mono/linker/blob/main/docs/design/trimmed-assemblies.md#assemblymetadataistrimmable-true.

* Add a condition to let projects opt out

A project just needs to set <SetIsTrimmable>false</SetIsTrimmable>, and
it won't get the trimmable metadata.

I also got rid of the SkipFrameworkAssemblyMetadata check since I don't
think the IsTrimmable metadata should be tied to whether we are building
a framework assembly, and this property was unused as far as I could tell.

Fix Socket.SendFile test: SliceBuffers_Success (#48457)

* do not expect a single receive

* use expected.Length

* enable test

Remove trimming attributes from any assembly. (#48424)

* Remove trimming attributes from any assembly.

These attributes are allowed to be in any assembly, for assemblies that are built using older TFMs where the attributes didn't exist.

Fix #48399

* Remove nullable attributes as well.

Normalized names SIMD_OP_LLVM_* -> SIMD_OP_ARM64_* (#48489)

Co-authored-by: Nathan Ricci <naricc@xam-emag-01.redmond.corp.microsoft.com>

[master] Update dependencies from dotnet/xharness (#48452)

[master] Update dependencies from dotnet/xharness

Expand PriorityQueue test coverage (#48420)

* make comparer field readonly

* abstract IComparer in PriorityQueue generic tests

does minor refactorings and adds new tests

* add PriorityQueue property tests

* address feedback

* add seed for deterministic runs

* apply var fixes in property tests

* replace FsCheck properties with theories

Removing myself from codeowners file (#48502)

In past I was involved in mono/debugger-agent.c, but not anymore, hence removing myself from CODEOWNERS file

[interp] Small interp tweaks (#48412)

* [interp] Fix overflow in PROFILE_INTERP log

* [interp] Replace LDFLDA of offset 0 field with MOV

Commonly used when accessing first field of valuetypes, eg span operations. Will help with cprop and remove redundant instruction.

* [interp] Remove redundant opcode