Bah si ça t'évite un code sensiblement plus long, oui c'est un bon choix ("propre" ça ne veut pas dire grand chose à ce niveau).

Par contre il me semble que GCC râle si tu n'ajoutes pas une paire de parenthèse autour. Il fait ça pour être sûr que tu veux bien utiliser une affectation, et que ce n'est pas un "while (i == i >> 1)" avec une faute de frappe.