Add debug_prints for bug...

Change-Id: Idc66ededc8e329737ed28b42e0f900be563166d0
Signed-off-by: Vyacheslav Cherkashin <v.cherkashin@samsung.com>

Add debug prints

Change-Id: Idb7693eca1a122f64ac9d2febc56b4514836afca
Signed-off-by: Vyacheslav Cherkashin <v.cherkashin@samsung.com>

[Tizen] Enable Tizen ASan runtime support

Enable libasansi.so support (libasan.so with switchable interceptors).

Change-Id: I9ee9b47b7beab55f036ffc0697ffab2583e9701c
Signed-off-by: Vyacheslav Cherkashin <v.cherkashin@samsung.com>

[Tizen] Implement ASan wrapper for Linux AMD64

Change-Id: I48446ce7c8771a4c75149512bb7d8a8cb3fae8e5
Signed-off-by: Vyacheslav Cherkashin <v.cherkashin@samsung.com>

[Tizen] Implement ASan wrapper for Linux ARM32

This commit implements wrappers that allow interception transitions
from managed to external unmanaged code (CIL -> native) and back
(native -> CIL). This allows enable/disable ASan during transitions.
Due to this, we sanitize only external code, which allows us to
achieve acceptable performance.

Change-Id: I53ecdc14d28f7210cd9e7f5bd4db0c8ef5ed81fc
Signed-off-by: Vyacheslav Cherkashin <v.cherkashin@samsung.com>

[Tizen] Implement detecting of sanitized libraries

Parse ".dynamic" section (ELF dynamic array tags) of the module being
added, find ".rel(a).plt" section and search it for presence of
'__asan_init' symbol.

Change-Id: Ie7cc4c818b791b5f00713b42ba15131325b8152c
Signed-off-by: Andrey Drobyshev <a.drobyshev@samsung.com>

add access(2) call before dlopening files

[Tizen] Precompile frequently used generic methods.

corbbtprof: set byte alignment for CORBBTPROF structures (#25816)

Fix patch fixes SIGBUG that occurs due to unaligned read/write

Fail to explicitly tail call on x86 unix. (#25032)

* Fail to explicitly tail call on x86 unix.

* Correctly return 100

* Correct return value

* Add noway assert in morphTailCall to avoid morphing slow tail calls on unix.

* Address feedback

Use regular static variable for EEConfig storage (#25479)

Fixes #25475

[Tizen] Add FEATURE_NGEN_RELOCS_OPTIMIZATIONS=true as default value for arm in spec file

[Tizen] Disable jithost arena cache

[Tizen] Fix x86 build error: 11 enumeration values not handled in switch

[Tizen] Fix gbs build

[Tizen] Add BuildTools for v3.0.0-preview7.19327.2

[Tizen] Disable IBC Logger as a default

- Profile information is collected by ibc logger.
  Hower it is not used and saved into profile file.
- The patch disables IBC logger which is enabled by default.
- It disables IBC logger only with ibclogger.h file.
  IBCLOGGER_ENABLED definition is only used in ibclogger files.

Fix a build error when IBCLOGGER_ENABLED is not defined (#25691)

- Even if DACCESS_COMPILE or CROSSGEN_COMPILE is defined,
  coreclr can be built without IBCLOGGER_ENABLED definition.

[Tizen] Define -DHAS_ADDRESS_SANITIZER for ASan build

This define turns on new and delete operators compatibility mode for build with
Address Sanitizer. It allows Tizen libasan to intercept operators and prevents
alloc-dealloc-mismatch errors.

Change-Id: If72b1d42dd99eaaf11d3cb1232006ee0e2e411c3
Signed-off-by: Slava Barinov <v.barinov@samsung.com>

[Tizen] Revert lockfiles creation.

directory creation for coreclr will be handled by dotnet-launcher
Revert previous patch.

[Tizen] Remove -DFEATURE_GDBJIT=TRUE from spec file

Change-Id: Ic0b8bfc8ab45284ecda5ab99e683224368e607f5

[Tizen] Update Openssl version 1.0 to 1.1

Change-Id: Id37364dd5dec17ab5a7b60e14df78a4967b3f3b8

[Tizen] Enable PGO for Linux/arm

[Tizen] seperate PIC and PIE to fix x86_64 build error

Revert "[Tizen] Exclude x86_64 build"

This reverts commit 546d7a1519cc81d98f2dadb5f2191c5a62e6f735.

Policy for arch build is cleared.
- In Public, unified builds each arch and they are independent.
- In SPIN, turn on and off each arch by necessity.

[Tizen] Exclude x86_64 build

Change-Id: I3c0fc85b839b55170fb177c4ac0cdd15dd063f5c

[Tizen] Add a library to retrieve memory regions for a coredump

This is needed to save a minicoredump of .NET applications to allows
reconstruct managed stack by lldb and SOS plugin.

Revert "[Tizen] create .dotnet directory to skel"

This reverts commit 26cdef6a45e6fb2c81fcbaf18be638f848525322.

[Tizen] update cscope files to be ignored

Change-Id: Ia0e1c6aa651c4fd88dc58b901729896a7a27681d

[Tizen] create .dotnet directory to skel

[Tizen] create "/tmp/.dotnet/lockfiles" directory while system bootup.

If "/tmp/.dotnet/lockfiles" directory does not exist, a directory is created when the first named mutex file is created.
The generated directory has the smack label of the process that created it.
Therefore, access deny occurs when another application tries to access the directory to create or use a named mutex.

To solve this problem, create "/tmp/.dotnet/lockfiles" directory in the tempfs while system boot up

[Tizen] Change paths to ilasm/ildasm

[Tizen] Rearrange executables among rpm pkgs

- `corerun` and `coreconsole` to `devel` pkg
- remove `extra` pkg and move symlinks in it.

Change-Id: I99cd26121d3c83242caf6f2418d686b3ffcc2828

[Tizen] Enable RELRO protection

[Tizen] Add crossgen and coreconsole to coreclr package

- The coreclr-extra package is going to be eliminated soon.
- Symlinks to crossgen and coreconsole are created temporarily for graceful migration

[Tizen] Provide symlinks to .NET version for compatibility

This commit will be reverted soon when all dependent modules watch the symlink at '/usr/share/dotnet.tizen/netcoreapp'.

Change-Id: Ib2ebf62a5788e225a9e3354f53fadde282708700

[Tizen] Add skipped testcase: superpmi

Skip JIT/superpmi/superpmicollect/superpmicollect.sh

[Tizen] Add skipped testcase on TM1

Lists:
JIT.jit64.opt.cse.hugeSimpleExpr1.hugeSimpleExpr1
GC.Regressions.v2.0-rtm.494226.494226.494226
GC.Scenarios.ServerModel.servermodel.servermodel
JIT.jit64.opt.cse.HugeField1.HugeField1
JIT.jit64.opt.cse.HugeField2.HugeField2
JIT.jit64.opt.cse.hugeexpr1.hugeexpr1

Fix OOPStackUnwinderX86::Unwind crash when Eip is invalid

Fix EECodeManager::GetAmbientSP on x86/Linux

[Tizen] Provide a symlink to installed Microsoft.NETCore.App version

The symlink is added to provide consistent access to installed Microsoft.NETCore.App version.
'/usr/share/dotnet.tizen/netcoreapp' is linked to '/usr/share/dotnet/shared/Microsoft.NETCore.App/2.1.0'.

[Tizen] Add -pie to linker option

There have been no -pie linker option.
This patch adds -pie linker option into crossgen(for tizen)

This originates from 0024-Add-pie-to-linker-option.patch

[Tizen] Enable -O3 optimization in clang 3.8

[Tizen] Add support for GBS

[Tizen] Add BuildTools 3.0.0-preview1-03617-02

[Tizen] Add prebuilt libicu-57.1 libraries

Merge pull request #25455 from dotnet/darc-release/3.0-7b624a21-2f92-409c-9902-75d5b2b62c49

[release/3.0] Update dependencies from dotnet/arcade

React to new compiler nullability warnings

Update dependencies from https://github.com/dotnet/arcade build 20190626.44

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19326.44
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19326.44
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19326.44
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19326.44

Merge pull request #25439 from dotnet-maestro-bot/merge/master-to-release/3.0

[automated] Merge branch 'master' => 'release/3.0'

Hardcode LKG version of ucrtbased.dll (#25444)

* Hardcode LKG version of ucrtbased.dll

* Only use hardcoded UCRT version when search path fails

* Add missing single-quotes

Fix EncoderNLS / DecoderNLS regression in reporting error index (#25397)

Also fixes incorrect asserts in the Encoding type

Merge branch 'release/3.0' into merge/master-to-release/3.0

[master] Update dependencies from dotnet/corefx (#25425)

* Update dependencies from https://github.com/dotnet/corefx build 20190626.2

- Microsoft.Bcl.AsyncInterfaces - 1.0.0-preview7.19326.2
- Microsoft.NETCore.Platforms - 3.0.0-preview7.19326.2
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview7.19326.2

* Reenable globalization tests

Tweak some annotations on EventRegistrationTokenTable<T> (#25386)

Delegate inspection API in the DBI (#25362)

* Add ICorDebugDelegateObjectValue interfaces
* Expose appropriate HR (CORDBG_E_UNSUPPORTED_DELEGATE)
* Partially DACize DelegateObject
* Add DacDbi method for delegate inspection

Merge pull request #25326 from BruceForstall/CrossPlatTestHost

Expand corefx testing in coreclr repo

Currently, corefx testing is done on Windows/x64.
Expand testing to include:
1. Linux/x64, Linux/arm, Linux/arm64, Windows/x86 platforms.
   Windows/arm should be added after corefx official builds
   publish tests and test manifest. Windows/arm64 should be
   added after corefx tests are published, and we have
   sufficient hardware.
2. All JIT stress modes that are used in normal coreclr testing.

Testing is split into 4 Azure DevOps Pipelines: coreclr-corefx,
coreclr-corefx-jitstress, coreclr-corefx-jitstressregs, and
coreclr-corefx-jitstress2-jitstressregs.

In addition, we now use the corefx meta-package Microsoft.Private.CoreFx.OOB
instead of a list of individual assemblies when constructing
the testhost.

Fix disabling wrong session on EventPipe::Shutdown (#25409)

- It's safe to call EventPipe::Disable during shutdown (eventpipe being disabled)

Fixes link to install SOS on linux (#25420)

Don't show AggressiveInlining items in exception stacks (#25408)

Reverting the ShouldRoundUp logic to not change for custom numeric format strings. (#25400)

Fix incorrect signature of QCall (#25419)

Fix LTTng filtering for runtime providers (#25374)

* LTTng filtering

* Fix Windows build, make IsInitialized check faster

* Cleanup/change default to no keywords/level set

* Keep the default to be the current policy - enable everything

* more cleanup

* No need to check for initialization

* Fix Windows build

Tell the compiler to emit nullable attributes for public API only (#25398)

Fix ReJIT failures (#25377)

emit rundown event for generic methods in R2R images (#25371)

* emit rundown event for generic methods in R2R images

* fix mac build

* Code Review Feedback

Expand corefx testing in coreclr repo

Currently, corefx testing is done on Windows/x64.
Expand testing to include:
1. Linux/x64, Linux/arm, Linux/arm64, Windows/x86 platforms.
   Windows/arm should be added after corefx official builds
   publish tests and test manifest. Windows/arm64 should be
   added after corefx tests are published, and we have
   sufficient hardware.
2. All JIT stress modes that are used in normal coreclr testing.

Testing is split into 4 Azure DevOps Pipelines: coreclr-corefx,
coreclr-corefx-jitstress, coreclr-corefx-jitstressregs, and
coreclr-corefx-jitstress2-jitstressregs.

In addition, we now use the corefx meta-package Microsoft.Private.CoreFx.OOB
instead of a list of individual assemblies when constructing
the testhost.

Remove the Crossgen-specific VSD flag from R2RDump (#25391)

Based on JanK's advice I have removed the VSD flag from R2RDump.
The flag should get never observed by the dump tool as it's just
an intermediate contract internal to the Crossgen ZAP logic.

Thanks

Tomas

Delete DeserializationBlockedException (#25393)

Contributes to dotnet/corefx#36723

Policheck comment adjustment

Update dependencies from https://github.com/dotnet/corefx build 20190624.8 (#25379)

- Microsoft.Bcl.AsyncInterfaces - 1.0.0-preview7.19324.8
- Microsoft.NETCore.Platforms - 3.0.0-preview7.19324.8
- Microsoft.Private.CoreFx.NETCoreApp - 4.6.0-preview7.19324.8

React to compiler respecting nullable attributes on properties

Update dependencies from https://github.com/dotnet/arcade build 20190624.24

- Microsoft.DotNet.Arcade.Sdk - 1.0.0-beta.19324.24
- Microsoft.DotNet.Build.Tasks.Feed - 2.2.0-beta.19324.24
- Microsoft.DotNet.Build.Tasks.Packaging - 1.0.0-beta.19324.24
- Microsoft.DotNet.Helix.Sdk - 2.0.0-beta.19324.24

Make sure EventPipeProvider::m_keywords is accurate after EventPipeConfiguration::Disable (#25358)

JitBench: Run PowerShell with an execution policy (#25259)

PowerShell is useless without this argument in default Windows configurations.

Map C Locale to Invariant (#25373)

ICU map the C locale to en-US_POSIX which has a non desired collation behavior as it doesn't support case insensitive comparison.
This change is to force mapping C locale to Invariant instead.

Updating the zapper to filter out some named intrinsics which are dependent on the ISAs supported by the target CPU. (#25365)

Fix StackChange value for OpCodes.Stelem (#25382)

Fixes dotnet/corefx#37955

Update dependencies from https://github.com/dotnet/core-setup build 20190624.03 (#25380)

- Microsoft.NETCore.App - 3.0.0-preview7-27824-03

Add pr.yml ci.yml internal.yml pipelines to /eng/pipelines (#25385)

Remove outputPath parameter for EventPipe IPC protocol (#25363)

Fix StackFrameIterator::IsValid check race (#25359)

* Fix StackFrameIterator::IsValid check race

During GC stress >= 4, there could be a race when we would compute
bRedirectedPinvoke as false, but before the condition of the following
_ASSERTE is evaluated, the thread that is being walked pushes a
ResumableFrame to the explicit frames stack of that thread in the GC
marker handler.

The fix to prevent this race is to evaluate all the conditions that
formed the bRedirectedPinvoke after the conditions in the _ASSERTE.

Add a note about ReadyToRun to ManagedNativeHeader

Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190624.3 (#25381)

- optimization.IBC.CoreCLR - 99.99.99-master-20190624.3
- optimization.PGO.CoreCLR - 99.99.99-master-20190624.3

Stop deallocating provider/events on shutdown (#25369)

Brick table (#25349)

Fix brick table logic to fix perf issue in several ASP.NET tests, remove #ifdef FFIND_OBJECT.

What I observed was that some GCs spent a lot of time in find_first_object called from find_object, which is called during stack scanning to find the containing object for interior pointers. A substantial fraction of generation 0 was being scanned, indicating that the brick table logic didn't work properly in these cases.

The root cause was the fact that the brick table entries were not being set in adjust_limit_clr if the allocation was satisfied from the free list in gen0 instead of newly allocated space. This is the case if there are pinned objects in gen0 as well.

The main fix is in adjust_limit_clr - if the allocation is satisfied from the freelist, seg is nullptr, the change is to set the bricks in this case as well if we are allocating in gen0 and the allocated piece is above a reasonable size threshold.

The bricks are not set always set during allocation - instead, when we detect an interior pointer during GC, we make the allocator set the bricks during the next GC cycles by setting gen0_must_clear_bricks. I changed the way this is handled for server GC (multiple heaps). We used to multiply the decay time by the number of heaps (gc_heap::n_heaps), but only applied it to the single heap where an interior pointer was found. Instead, I think it's better to instead set gen0_must_clear_bricks for all heaps, but leave the decay time unchanged compared to workstation GC.

Maoni suggested to remove the #ifdef FFIND_OBJECT - interior pointers are not going away, so the #ifdefs are unnecessary clutter.

Addressed code review feedback:
 - add parentheses as per GC coding conventions
 - use max instead of if-statement
 - merge body of for-loop over all into existing for-loop

Fix broken ETW events (#25364)

Make s_numberOfSessions Volatile as advertisement to other devs that the data is being used across multiple threads and a portion of that usage is lock-free (#25366)

Events with keyword 0 needs to be fired (#25361)

Address last round of nullable feedback - S.Resources.ResourceManager, System.Reflection.Emit and ILGeneration (#25318)

* Address nullable feedback for System.Resources.ResourceManager

* Address nullable feedback for System.Reflection.Emit.ILGeneration

* Address nullable feedback for System.Reflection.Emit

* PR Feedback and clean-up redundant casts

Add /eng/pipelines/*.yml (#25325)

Throw platform unsupported error for windows arm32 varargs (#25320)

* Throw unsupported platform for native vcarargs on Arm32 Windows

* Re-enable baseservices vararg test

Updating Dragon4 to ensure the number buffer always provides a significant digit if one exists. (#25096)

* Updating Dragon4 to ensure the number buffer always provides a significant digit if one exists.

* Changing System.Number.RoundNumber to not round up floating-point numbers.

* Re-enabling the RealFormatterTestsBase CoreFX tests

* Updating Number.RoundNumber to take a isCorrectlyRounded parameter and to use IEEE compliant rounding for floating-point numbers.

* Change SinglePrecisionCustomFormat to 7, ensuring it matches the value used in netcoreapp2.1

Switch from IndexOf(string) to Contains(string) for better readability and performance (#38757)

* Fix performance IndexOf(string) -> Contains(string)

* Make comparison ordinal

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

Make Vector*.ToString() Culture invariant (#25343)

* Remove IFormattable
* Hardcode characters
* Add tests

Multiple bug fixes (#25308)

- Fixes EventPipe does not properly dispose of itself on an active session error #25228
  - On EventPipeSession::ThreadProc, if an error occurs on the IPC streaming, then EventPipe::Disable will be invoked.
- Fixes Prevent EventPipe Sessions IDs from being reused on disconnect #25229

Remove AggressiveOptimization flag from System.Text (#25356)

Based on conversation with Levi, I don't think this flag is needed.

The flag currently controls two things:
* Disqualifies the method from precompilation
* Disables Tier-0 JIT for the method

Removing the attribute allows us to precompile some of the methods (not all, because of `Vector<T>`). It measurably helps improve startup time because many of the attributed methods are complex and take a while to compile.

Once tier-1 JIT kicks in, the generated code should be identical to what we were getting before.

JIT: see if jmp offset will fit in 32 bit displacement (#25348)

On x86 we can always reach any IAT_PVALUE entry via a 32 bit indirect
jump, and proper handling of jmp epilogs depends on this. So check if the
target address is reachable in 32 bits, and if so, use the jmp [addr] form
on x86, and the jmp [rip + disp] form for x64.

Fixes #25345
Fixes #25346
Undoes the assertion change from #25302
Fixes #25286

Update R2R format doc to 3.0 (#25297)

I have updated the various enumerations and tables. I didn't try
to go overboard w.r.t. understanding tables I'm not yet familiar
with, most notably the diagnostic, inlining and profiling tables.

Update dependencies from https://dev.azure.com/dnceng/internal/_git/dotnet-optimization build 20190624.1 (#25354)

- optimization.IBC.CoreCLR - 99.99.99-master-20190624.1
- optimization.PGO.CoreCLR - 99.99.99-master-20190624.1