jhenderson added a comment. The code change looks fine now, but there's still no context in the diff. Are you going to add unit tests too? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62838/new/ https://reviews.llvm.org/D62838