ashermancinelli wrote: > Should not we instead check for `getValue` being `nullptr` at the places where it is used? It seems to be more generic than checking for volatile explicitly. I think you're right. https://github.com/llvm/llvm-project/pull/140781