zturner added a subscriber: eax. zturner added a comment. It actually looks wrong to me to *not* have the side effect. If this is due to some warning, then the increment needs to be moved after the assert Repository: rL LLVM https://reviews.llvm.org/D46620