RFR (XXL): 8003250: SPARC: move MacroAssembler into separate file
Christian Thalinger
christian.thalinger at oracle.com
Thu Dec 6 09:57:39 PST 2012
Thank you, Vladimir. -- Chris
On Dec 5, 2012, at 7:34 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> 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