Request for Review(S): Ifop optimization for c1
Roland Westrelin
roland.westrelin at oracle.com
Thu Oct 14 04:30:03 PDT 2010
Hi Rémi,
> A minor comment from an amateur.
> In c1_Optimizer.cpp, in CE_Eliminator::make_ifop:
>
> Constant* x_const = x->as_Constant();
>
> should be computed only in the else branch and not before
>
> if (x_ifop != NULL)
Thanks for the comment. I've updated the webrev:
http://cr.openjdk.java.net/~roland/6991577/webrev.01/
Roland.
More information about the hotspot-compiler-dev
mailing list