RFR (XXL): 8003250: SPARC: move MacroAssembler into separate file
Vladimir Kozlov
Vladimir.Kozlov at oracle.com
Wed Dec 5 19:34:36 PST 2012
Good.
Vladimir
On 12/05/12 19:17, Christian Thalinger wrote:
>
> On Dec 5, 2012, at 5:22 PM, John Rose<john.r.rose at oracle.com> wrote:
>
>> On Dec 5, 2012, at 4:22 PM, Christian Thalinger wrote:
>>
>>> [Prologue: As for 8003240 this patch is XXL but it's mostly verbatim copying. I say mostly because this time I had to move some methods from Assembler to MacroAssembler to get it working. Some methods were removed (e.g. test_v9, test_v8_onlys) because they were not used at all.]
>>
>> I did not verify the verbatim copying, but assuming that there are no extra changes hiding in the moved lines, it is good.
>
> Thank you, John.
>
> By Vladimir's request I removed macroAssembler.hpp includes where we include macroAssembler.inline.hpp and added the include to macroAssembler.inline.hpp:
>
> http://cr.openjdk.java.net/~twisti/8003250/src/share/vm/asm/macroAssembler.inline.hpp.udiff.html
>
> -- Chris
>
>>
>> — John
>
More information about the hotspot-compiler-dev
mailing list