https://github.com/jh7370 commented: Basically looks fine to me. I've suggested one small tweak to the test. If we find the error message to get in the way of a genuine use case, we can look to modify/remove it in the future. https://github.com/llvm/llvm-project/pull/118741