RFR(S) 8032656: Tag the MachSpillCopies with purpose information (original) (raw)

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Tue Jan 28 02:50:53 PST 2014

Niclas,

Why didn't you introduce an enum instead and pass the reason to constructor? Introducing 13 subclasses just to overload Name() method in debug builds looks like an overkill to me.

Best regards, Vladimir Ivanov

On 1/28/14 2:07 PM, Niclas Adlertz wrote:

Hi all,

When debugging or visualizing spills/splits/moves in C2 it's hard to know what caused an insertion of a certain MachSpillCopy. I've made this easier by creating subclasses of the MachSpillCopyNode. webrev: http://cr.openjdk.java.net/~adlertz/JDK-8032656/webrev00/ bug: https://bugs.openjdk.java.net/browse/JDK-8032656 Kind Regards, Niclas Adlertz

More information about the hotspot-compiler-dev mailing list