dsanders accepted this revision. dsanders added a comment. Yep, LGTM. One nit: We could avoid introducing the Optional<> by testing for LLT.isValid() instead of testing for whether the Optional has a value Repository: rL LLVM https://reviews.llvm.org/D33084