Hardcode84 wrote: > just to confirm, this change is NFC right? Should be NFC, I removed `return static_cast<unsigned>(NVVM::NVVMMemorySpace::Generic);` after the `llvm_unreachable` but it would never be executed anyway. https://github.com/llvm/llvm-project/pull/172156