RFR (S) 8204195: Clean up macroAssembler.inline.hpp and other inline.hpp files included in .hpp files

Jiangli Zhou jiangli.zhou at oracle.com
Thu May 31 23:52:31 UTC 2018


The changes look good to me.

Thanks,
Jiangli

> On May 31, 2018, at 4:10 PM, coleen.phillimore at oracle.com wrote:
> 
> Summary: Moved macroAssembler.inline.hpp out of header file and distributed to .cpp files that included them: ie. c1_MacroAssembler.hpp and interp_masm.hpp. Also freeList.inline.hpp and allocation.inline.hpp.
> 
> open webrev at http://cr.openjdk.java.net/~coleenp/8204195.01/webrev
> bug link https://bugs.openjdk.java.net/browse/JDK-8204195
> 
> Tested with mach5 hs-tier1,2 on Oracle platforms: linux-x64, solaris-sparcv9, macosx-x64 and windows-x64.  Also tested zero and aarch64 fastdebug builds, and linux-x64 without precompiled headers.   Please test other platforms, like arm32, ppc and s390!  I think these are the last platform dependent inline files that are included by .hpp files.
> 
> Thanks,
> Coleen



More information about the hotspot-dev mailing list