Request for reviews (XS): 7079317: Incorrect branch's destination block in PrintoOptoAssembly output
vladimir.kozlov at oracle.com
Mon Aug 15 10:50:10 PDT 2011
Node::size() for branches calls code in scratch_emit_size() which resets label
and block. An other solution for this problem would be save/restore label and
block in scratch_emit_size() but it would require a lot more code changes.
Tom Rodriguez wrote:
> I don't understand how calling insts_size and Node::size causes a bug. What am I missing?
> On Aug 15, 2011, at 8:58 AM, Vladimir Kozlov wrote:
>> 7079317: Incorrect branch's destination block in PrintoOptoAssembly output
>> After changes for 7063629 PrintoOptoAssembly output shows all branches have B0 as destination block.
>> Remove unneeded debug verification code which overwrites label and block information for branches. There are other checks there which verify that code size was not changed.
More information about the hotspot-compiler-dev