[llvm] [VPlan] Fix header phi VPInstruction verification. NFC (PR #151472)

David Sherwood via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 31 02:01:53 PDT 2025


================
@@ -80,8 +80,8 @@ bool VPlanVerifier::verifyPhiRecipes(const VPBasicBlock *VPBB) {
       NumActiveLaneMaskPhiRecipes++;
 
     if (IsHeaderVPBB && !isa<VPHeaderPHIRecipe, VPWidenPHIRecipe>(*RecipeI) &&
-        !isa<VPInstruction>(*RecipeI) &&
-        cast<VPInstruction>(RecipeI)->getOpcode() == Instruction::PHI) {
+        !(isa<VPInstruction>(*RecipeI) &&
----------------
david-arm wrote:

But looking at VPPredInstPHIRecipe it seems unlikely to be in the header, right? So perhaps there is no way to write a test case that proves this was broken.

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


More information about the llvm-commits mailing list