https://github.com/phoebewang approved this pull request. LGTM. I think we need to add Doxygen to make arguments clear to user since it's easily errorprone. https://github.com/llvm/llvm-project/pull/76389