In email discussion with [~vaivanov], he pointed out that we do have to deoptimize all nmethods with late attach because adding evol_methods dependencies also adds the Method* to the metadata. This Method* is what redefinition uses to find if there are old methods in the nmethod.
So we still need to deoptimize all the nmethods if evol_method dependencies aren't enabled in the ON_LOAD phase.
Backing out change and redoing with simple removal of the function we no longer need. I think this is easiest.