[PATCH] D142885: [VPlan] Allow building a VPlan to may fail.

[Thorsten via Phabricator via llvm-commits]

tschuett added a comment.

In D142885#4209221 <https://reviews.llvm.org/D142885#4209221>, @fhahn wrote:

> Rename to tryToBuildVPlanWithVPRecipes as suggested, thanks!

I was never questioning the move. I just wanted to note that I prefer std::nullopt over nullptr to denote failure.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142885/new/

https://reviews.llvm.org/D142885