RFR 8080325 SuperWord loop unrolling analysis

Roland Westrelin roland.westrelin at oracle.com
Tue Jun 9 13:02:26 UTC 2015

> http://cr.openjdk.java.net/~mcberg/8080325/webrev.01/

Since you’re touching that code, can you also fix the coding style in:

In loopTransform.cpp

644 bool IdealLoopTree::policy_unroll( PhaseIdealLoop *phase ) {

In loopnode.hpp

462   bool policy_unroll( PhaseIdealLoop *phase );

(no spaces after/before the parenthesis)

In loopnode.hpp:

164   int slp_maximum_unroll_factor;

should be: _slp_maximum_unroll_factor

248   int  slp_max_unroll()                      { return slp_maximum_unroll_factor; }

could be: int  slp_max_unroll() const {

 464   // Return TRUE or FALSE if the loop analyzes to map to a maximal
 465   // superword unrolling for vectorization.
 466   void policy_unroll_slp_analysis(CountedLoopNode *cl, PhaseIdealLoop *phase, int future_unroll_ct);

The comment says the function returns something but it doesn’t return anything.

I don’t see slp_maximum_unroll_factor being initialized to a default value. Isn’t there a risk it’s not set when we read it?

Otherwise, I think it’s good.


More information about the hotspot-compiler-dev mailing list