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