[9] RFR (S): 8152773: C2: LoadNode properties aren't preserved when converting between signed/unsigned variants

John Rose john.r.rose at oracle.com
Fri Mar 25 19:28:11 UTC 2016


On Mar 25, 2016, at 11:51 AM, Vladimir Ivanov <vladimir.x.ivanov at oracle.com> wrote:
> 
> I think the motivation was to avoid massive change in all Load*Nodes. LoadNode::make constructs a proper flavor and then sets mismatched/unaligned flags.
> 
>> I think they are separate booleans for the usual reason:  They were introduced at different times.
> 
> Do you think we should pack them?

Yes, I think they should be packed and pushed into the constructor.

But I see it's a larger change than I hoped.  Not an on-the-fly cleanup.

— John
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20160325/662905aa/attachment.html>


More information about the hotspot-compiler-dev mailing list