andykaylor wrote: > Should we avoid returning optional when `FailureOr` conveys that information better? That's a good idea, but for this change I'd like to leave it as it is/was if everyone is OK with that. https://github.com/llvm/llvm-project/pull/233639