david-arm wrote: Also, I think if this patch is fixing a bug (which you implied from the commit message) then it probably shouldn't have NFC in the title or commit message? That sounds like a functional change. https://github.com/llvm/llvm-project/pull/118393