Andrew Dinn adinn at redhat.com
Wed Dec 16 17:28:15 UTC 2015

On 16/12/15 17:06, Coleen Phillimore wrote:
> Andrew,  Thank you!  The relevant files are;
> R src/cpu/aarch64/vm/bytecodeInterpreter_aarch64.cpp
> R src/cpu/aarch64/vm/bytecodeInterpreter_aarch64.hpp
> R src/cpu/aarch64/vm/bytecodeInterpreter_aarch64.inline.hpp
> R src/cpu/aarch64/vm/cppInterpreterGenerator_aarch64.hpp

Yes, removing those files is fine.

> My patch also consolidates these functions so these files are removed
> also (as well as the conditional inclusion in
> interpreter/templateInterpreterGenerator.hpp)
> R src/cpu/aarch64/vm/interpreterGenerator_aarch64.hpp
> R src/cpu/aarch64/vm/templateInterpreterGenerator_aarch64.hpp

Do you mean that the declarations in the latter two files have all been
moved up to generic headers? If so then that is also fine.


Andrew Dinn

More information about the hotspot-dev mailing list