RFR 8003426: Remove UseFastAccessors and UseFastEmptyMethods except for zero

[Vladimir Kozlov]

Nice cleanup. Looks good to me.

Thanks,
Vladimir

On 8/7/14 9:17 AM, Coleen Phillimore wrote:
> Summary: These options have been long disabled in Xmixed mode because
> they prevent these small methods from being inlined and are subject to
> bit rot, and we don't need more macro assembler code to maintain and
> change if the constant pool cache format changes.
>
> I ran specjvm98 -Xint and there wasn't a performance improvement with
> these flags.  Zero reportedly needs these flags for performance, so I
> left them in for zero.  I did build zero with some other changes to make
> zero build (not included in this patch) though.  There are PPC64 changes
> which Volker offered to test.  Also built c++ interpreter on x86 and
> sparc and got same runtime error as already reported.
>
> generate_method_entry was at the wrong level of inheritance so I fixed
> that and made it common.  There was also some duplicate comment about
> frame layout and calling conventions (not sure it it's correct) that I
> either removed because it's in frame_x86.hpp or moved to before
> generate_fixed_frame() where it makes more sense.
>
> Tested with JPRT, nsk, jck and hotspot jtreg tests.
>
> open webrev at http://cr.openjdk.java.net/~coleenp/8003426/
> bug link https://bugs.openjdk.java.net/browse/JDK-8003426
>
> Thanks,
> Coleen
>
>