Simplify ClassGetTS interpreter implementation
commitfd774552793c9c422903ef37a1d5646c09fef73c
authorJan Oravec <jan@fb.com>
Thu, 5 Sep 2019 08:19:34 +0000 (5 01:19 -0700)
committerHhvm Bot <hhvm-bot@users.noreply.github.com>
Thu, 5 Sep 2019 08:21:30 +0000 (5 01:21 -0700)
tree7cdffe8550a5c35d4410bff99cc850102f582ff0
parent9b5f318e044e782e12f765970261881d4c6d2688
Simplify ClassGetTS interpreter implementation

Summary:
Do not mangle class name together with generics, just to pass it to
lookupClsRef(), which forwarded it to Unit::loadClass(), which unmangled it and
used just the original class name.

Instead, call the Unit::loadClass() with the original class name.

This was the last usage of mangleReifiedName(), i.e. the last place
in the codebase constructing '$$name$$<generics>' reified names.

The next diff will remove all the complexity related to reading these names.

We should probably also kill the reified generics table that tries to make
reified types static, but that's for another diff...

Reviewed By: ricklavoie

Differential Revision: D17200413

fbshipit-source-id: 75d5d5898aca184321036aed2213bf663abff28c
hphp/runtime/vm/bytecode.cpp
hphp/runtime/vm/unit-util.h