CppInterpreter

[Andrew Dinn]

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.

regards,


Andrew Dinn
-----------