aeubanks wrote: as mentioned in https://github.com/llvm/llvm-project/pull/76517, we should consider changing the default to required instead of optional, but this lgtm for now https://github.com/llvm/llvm-project/pull/76516