CQ fix for generic handle lookup in GenericHandleWorker
authorChris McKinsey <chrismck@microsoft.com>
Thu, 2 Apr 2015 20:09:44 +0000 (13:09 -0700)
committerChris McKinsey <chrismck@microsoft.com>
Thu, 2 Apr 2015 20:23:31 +0000 (13:23 -0700)
commitdf886a58dd4594ce942fa7ee78f6bff8f472cb5c
tree0d86ced180a0e3b779ac0356b842843c0ccc6546
parentcf104b93a168f7aaff7c14113b2f80106bbcc405
CQ fix for generic handle lookup in GenericHandleWorker

When looking up the runtime handle in the generic handle cache for a methodtable
we get the declaring methodtable in the hierarchy and then lookup the handle in
the cache from that methodtable. When found we should insert the handle back into
the table using the original methodtable as the key and not the declaring MT
so that later lookups are faster. Inserting the handle back into the table under
the original key was a noop.

Desktop DDR and JITSH testing clean. The test case in Github issue #55 is now
3x faster. Benchmarked roslyn performance which shows no change in performance.
src/vm/jithelpers.cpp