[llvm] Revert "[GVN][NFC] Use early return in phiTranslateImpl() (#149268)" (PR #149270)

[Madhur Amilkanthwar via llvm-commits]

https://github.com/madhur13490 created https://github.com/llvm/llvm-project/pull/149270

This reverts commit 1d398a96dc6b58d15d289c71e2d9f229a0ba719b.

>From b49192f005b9a69b6bbd267c56c2b787eebc270e Mon Sep 17 00:00:00 2001
From: Madhur Amilkanthwar <madhura at nvidia.com>
Date: Thu, 17 Jul 2025 01:10:11 -0700
Subject: [PATCH] Revert "[GVN][NFC] Use early return in phiTranslateImpl()
 (#149268)"

This reverts commit 1d398a96dc6b58d15d289c71e2d9f229a0ba719b.
---
 llvm/lib/Transforms/Scalar/GVN.cpp | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/llvm/lib/Transforms/Scalar/GVN.cpp b/llvm/lib/Transforms/Scalar/GVN.cpp
index affae41ed2c83..8bff458f88bb9 100644
--- a/llvm/lib/Transforms/Scalar/GVN.cpp
+++ b/llvm/lib/Transforms/Scalar/GVN.cpp
@@ -2367,15 +2367,12 @@ uint32_t GVNPass::ValueTable::phiTranslateImpl(const BasicBlock *Pred,
   // See if we can refine the value number by looking at the PN incoming value
   // for the given predecessor.
   if (PHINode *PN = NumberingPhi[Num]) {
-    if (PN->getParent() != PhiBlock)
-      return Num;
-
-    for (unsigned I = 0; I != PN->getNumIncomingValues(); ++I) {
-      if (PN->getIncomingBlock(I) != Pred)
-        continue;
-      if (uint32_t TransVal = lookup(PN->getIncomingValue(I), false))
-        return TransVal;
-    }
+    if (PN->getParent() == PhiBlock)
+      for (unsigned I = 0; I != PN->getNumIncomingValues(); ++I)
+        if (PN->getIncomingBlock(I) == Pred)
+          if (uint32_t TransVal = lookup(PN->getIncomingValue(I), false))
+            return TransVal;
+    return Num;
   }
 
   if (BasicBlock *BB = NumberingBB[Num]) {