54-line if condition in gcc's reload.c
Why put logical operator at the start and not the end of each line?<p>I.e., this style (used in this case)<p><pre><code>      && (CONSTANT_P (SUBREG_REG (in))
	  || GET_CODE (SUBREG_REG (in)) == PLUS
	  || strict_low
	  || (((REG_P (SUBREG_REG (in))
</code></pre>
versus this style:<p><pre><code>      (CONSTANT_P (SUBREG_REG (in)) ||
	  GET_CODE (SUBREG_REG (in)) == PLUS ||
	  strict_low ||
	  (((REG_P (SUBREG_REG (in)) &&
</code></pre>
I don't have a personal preference here, just looking for any practical pros and cons I may not be aware of.<p>More on topic, I don't see why keep such a big conditional and not move it out to its own function(s) (but this has been asked already in this thread).