RFR (S) 8204195: Clean up macroAssembler.inline.hpp and other inline.hpp files included in .hpp files
coleen.phillimore at oracle.com
coleen.phillimore at oracle.com
Fri Jun 1 10:35:28 UTC 2018
Thanks Jiangli!
Coleen
On 5/31/18 7:52 PM, Jiangli Zhou wrote:
> 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