skan added a comment. Is this a NFC change or a bug fix? If NFC, shouldn't we use assert? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157948/new/ https://reviews.llvm.org/D157948