Not generally against this change. Just the verbally expressed arguments...
- Saving a line of source code is not an optimization on its own. (besides that can be done by deleting the linebreak).
- Leading to less assembler => even less a reason. That is why we don't code in asm.
+ Making it clear, that the condition is always wrong before the first iteration of the loop => IMHO that is a reason.
"less assembler"
In some bottle neck cases, this may be needed, until the compiler can do it.
Though, then it should be "faster asm" as the reason, which is very hard to establish given the amount of targets there are.
"less assembler" may sometimes be slower.
In this case, it will likely not be slower. If it will be faster is a different question.