george.burgess.iv added a comment. I don't know this code well enough to LGTM this, but can you talk a bit about why, please? This side-effect-inside-an-assert looks sane to me. Repository: rL LLVM https://reviews.llvm.org/D46620