Fixup: wrap an intrinsic node which is used as a mask op for Conditio…

…nalSelect in another ConditionalSelect if required
dotnet · Jun 14, 2024 · 1a2329e · 1a2329e
1 parent 7827d1d
commit 1a2329e
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/coreclr/jit/lowerarmarch.cpp b/src/coreclr/jit/lowerarmarch.cpp
@@ -1303,7 +1303,8 @@ GenTree* Lowering::LowerHWIntrinsic(GenTreeHWIntrinsic* node)
         {
             GenTree* user = use.User();
             // Wrap the intrinsic in ConditionalSelect only if it is not already inside another ConditionalSelect
-            if (!user->OperIsHWIntrinsic() || (user->AsHWIntrinsic()->GetHWIntrinsicId() != NI_Sve_ConditionalSelect))
+            if (!user->OperIsHWIntrinsic() || (user->AsHWIntrinsic()->GetHWIntrinsicId() != NI_Sve_ConditionalSelect) ||
+                (HWIntrinsic(user->AsHWIntrinsic()).op1 == node))
             {
                 CorInfoType simdBaseJitType = node->GetSimdBaseJitType();
                 unsigned    simdSize        = node->GetSimdSize();