RFR (S): 8029025: PPC64 (part 203): opto: Move static _in_dump_cnt to Compile object.

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Nov 22 08:57:38 PST 2013


Looks good. Compile::current() is expensive but it is only for debugging. I will push it today.

Thanks,
Vladimir

On 11/22/13 7:39 AM, Lindenmaier, Goetz wrote:
> Hi,
>
> I prepared a webrev containing some debugging support.
> http://cr.openjdk.java.net/~goetz/webrevs/8029025-0-ndmp/
>
> To allow some special cases when dumping debug information about IR
> nodes, _in_dump_cnt can be increased. Unfortunately this is a global
> field.  If running with more than one compiler thread races can happen.
> As consequence, dumping crashes e.g. in MachProjNode::adr_type().
>
> This change moves the field to the Compile object.
>
> It also introduces the compiler oracle 'option' feature for PrintAssembly.
>
> This is not a mandatory part of the port, but we find this very useful
> if debugging the compiler.
>
> Please review and test this change
>
> Best regards,
>    Goetz.
>


More information about the ppc-aix-port-dev mailing list