michaelmaitland wrote: > LGTM with minor nit. @michaelmaitland sorry, I can't find the "minor nit". Which one do you mean? I think you are addressing it. It was the `//` -> `///` https://github.com/llvm/llvm-project/pull/118097