https://github.com/erichkeane approved this pull request. This change looks right to me! I don't know if there is value in a signed-test instead of unsigned, but a couple more tests might be nice anyway. https://github.com/llvm/llvm-project/pull/89254