Request for reviews (XXL): 6961690: load oops from constant table on SPARC

[Vladimir Kozlov]

Looks good. Add the assert into MachConstantBaseNode::emit sparc code.

Thanks,
Vladimir

Christian Thalinger wrote:
> On Nov 25, 2010, at 4:57 PM, Christian Thalinger wrote:
>>> I don't understand how MachConstantBaseNode::size() is not called
>>> on sparc where there is code generated for it? You need the size.
>>> It seems, you have to calculate it manually.
>> SPARC does not use Compile::Shorten_branches and that's the only place where I got calls to size.
> 
> I implemented MachConstantBaseNode::size on SPARC too.
> 
>>> I think you don't need to add_req() in constructor and then use
>>> set_req(root) since new MachConstantBase node is only created in
>>> Compile::mach_constant_base_node() so you can add_req(root) there
>>
>> OK.
>>
>> I will send out a review later or tomorrow.
> 
> ...or today:
> 
> http://cr.openjdk.java.net/~twisti/6961690/webrev.04/
> 
> I removed the additional pipe_class'es on SPARC again.  That should be the final version.
> 
> -- Christian