[llvm] [VPlan] Make simplifyRecipe more like InstCombine (PR #212968)

[Ramkumar Ramachandra via llvm-commits]

================
@@ -1658,7 +1613,16 @@ void VPlanTransforms::simplifyRecipes(VPlan &Plan) {
   for (VPBasicBlock *VPBB : VPBlockUtils::blocksOnly<VPBasicBlock>(RPOT)) {
     for (VPRecipeBase &R : make_early_inc_range(*VPBB))
       if (auto *Def = dyn_cast<VPSingleDefRecipe>(&R))
-        simplifyRecipe(Def);
+        if (VPValue *New = simplifyRecipe(Def)) {
+          if (New != Def) {
----------------
artagnon wrote:

I don't think simplifyRecipe is an identity function on no-simpl? Doesn't it return nullptr on no-simpl, or a different value in all cases?

https://github.com/llvm/llvm-project/pull/212968