RFR: 8041794: remove bytecodes_<arch>.{cpp,hpp} files

[Lev Priima]

Thanks Coleen and Christian !

Lev

On 06/17/2014 09:37 PM, Christian Thalinger wrote:
> Looks good.  Thanks for picking that one up.
>
> On Jun 17, 2014, at 9:08 AM, Lev Priima <lev.priima at oracle.com> wrote:
>
>> Hi,
>>
>> Problem:
>> We don't have any platform specific bytecodes and these files are empty:
>> hs-comp/hotspot$ find -name bytecodes_\* | grep -v "/.hg/"
>> ./src/cpu/ppc/vm/bytecodes_ppc.cpp
>> ./src/cpu/ppc/vm/bytecodes_ppc.hpp
>> ./src/cpu/sparc/vm/bytecodes_sparc.cpp
>> ./src/cpu/sparc/vm/bytecodes_sparc.hpp
>> ./src/cpu/x86/vm/bytecodes_x86.cpp
>> ./src/cpu/x86/vm/bytecodes_x86.hpp
>> ./src/cpu/zero/vm/bytecodes_zero.cpp
>> ./src/cpu/zero/vm/bytecodes_zero.hpp
>>
>> .hpp's are only included in bytecodes.hpp:
>> hs-comp/hotspot$ grep -rn \"bytecodes_\* src
>> src/share/tools/IdealGraphVisualizer/Data/src/com/sun/hotspot/igv/data/serialization/Parser.java:79: public static final String BYTECODES_ELEMENT = "bytecodes";
>> src/share/vm/interpreter/bytecodes.hpp:292:# include "bytecodes_x86.hpp"
>> src/share/vm/interpreter/bytecodes.hpp:295:# include "bytecodes_sparc.hpp"
>> src/share/vm/interpreter/bytecodes.hpp:298:# include "bytecodes_zero.hpp"
>> src/share/vm/interpreter/bytecodes.hpp:301:# include "bytecodes_arm.hpp"
>> src/share/vm/interpreter/bytecodes.hpp:304:# include "bytecodes_ppc.hpp"
>>
>> All bytecodes_<arch>.cpp files have empty implementations of their functions.
>> Function:
>> src/share/vm/interpreter/bytecodes.hpp:335: static Code pd_base_code_for(Code code);
>> with empty implementation in each platform-specific bytecodes_* file is never called.
>>
>>
>> Solution:
>> I've removed these files and include directives of them. Also I removed two function declarations and one call-statement of pd_initialize().
>>
>> Bug:
>> https://bugs.openjdk.java.net/browse/JDK-8041794
>>
>> Webrev:
>> http://cr.openjdk.java.net/~iignatyev/lpriima/8041794/webrev.00/
>>
>> Testing:
>> Jprt build and tests
>>
>> -- 
>> Lev
>>