arsenm accepted this revision. arsenm added a comment. This revision is now accepted and ready to land. LGTM. Could we just get rid of the argumentless one and use a default argument? https://reviews.llvm.org/D26994