Fix issue with excessive memory use by debug and checked jits
authorAndy Ayers <andya@microsoft.com>
Wed, 17 Feb 2016 07:46:16 +0000 (23:46 -0800)
committerAndy Ayers <andya@microsoft.com>
Wed, 17 Feb 2016 08:02:18 +0000 (00:02 -0800)
commitf35ed1fe296249669d764d5a143530c1e39dfb84
treea967a3276369a1d27f79b17cfc953a483ded2e90
parentef12c45733ef9ea6fb6a9039df5fbf7ad53e6264
Fix issue with excessive memory use by debug and checked jits

In dotnet/coreclr#3051 the jit started calling `eeGetClassFullName` for both
caller and callee handles even when dumping was disabled, passed
this information to the logging messages where previously the caller
and callee were identified via other context, and prepared the full
name dump artifacts for all inline decisions rather than the selective
cases handled previously.

This change caused excessive memory use in some tests.

This change makes the calls to `eeGetClassFullName` conditional on
whether dumping is enabled, and does not require these calls for logging
messages.

Commit migrated from https://github.com/dotnet/coreclr/commit/5e852304f2eff62533e103d706a2604a1e82ab17
src/coreclr/src/jit/compiler.cpp