diff --git a/src/tests/JIT/HardwareIntrinsics/Arm/Shared/SveGatherVectorByteOffsetFirstFaulting.template b/src/tests/JIT/HardwareIntrinsics/Arm/Shared/SveGatherVectorByteOffsetFirstFaulting.template index 6cb1706098a50..e7b466bfb201a 100644 --- a/src/tests/JIT/HardwareIntrinsics/Arm/Shared/SveGatherVectorByteOffsetFirstFaulting.template +++ b/src/tests/JIT/HardwareIntrinsics/Arm/Shared/SveGatherVectorByteOffsetFirstFaulting.template @@ -60,17 +60,7 @@ namespace JIT.HardwareIntrinsics.Arm test.RunStructFldScenario(); // Validates using inside ConditionalSelect with value falseValue - // Currently, using this operation in ConditionalSelect() gives incorrect result - // when falseReg == targetReg because this instruction uses Pg/Z to update the targetReg - // instead of Pg/M to merge it. As such, the value of falseReg is lost. Ideally, such - // instructions should be marked similar to RMW (a different flag name) to make sure that - // we do not assign falseReg/targetReg same. Then, we would do something like this: - // - // ldnf1sh target, pg/z, [x0] - // sel mask, target, target, falseReg - // - // This needs more careful thinking, so disabling it for now. - // test.ConditionalSelect_FalseOp(); + test.ConditionalSelect_FalseOp(); // Validates using inside ConditionalSelect with zero falseValue test.ConditionalSelect_ZeroOp(); diff --git a/src/tests/JIT/HardwareIntrinsics/Arm/Shared/SveGatherVectorFirstFaultingIndices.template b/src/tests/JIT/HardwareIntrinsics/Arm/Shared/SveGatherVectorFirstFaultingIndices.template index 381da3c5a5133..28aa98b11be70 100644 --- a/src/tests/JIT/HardwareIntrinsics/Arm/Shared/SveGatherVectorFirstFaultingIndices.template +++ b/src/tests/JIT/HardwareIntrinsics/Arm/Shared/SveGatherVectorFirstFaultingIndices.template @@ -60,17 +60,7 @@ namespace JIT.HardwareIntrinsics.Arm test.RunStructFldScenario(); // Validates using inside ConditionalSelect with value falseValue - // Currently, using this operation in ConditionalSelect() gives incorrect result - // when falseReg == targetReg because this instruction uses Pg/Z to update the targetReg - // instead of Pg/M to merge it. As such, the value of falseReg is lost. Ideally, such - // instructions should be marked similar to RMW (a different flag name) to make sure that - // we do not assign falseReg/targetReg same. Then, we would do something like this: - // - // ldnf1sh target, pg/z, [x0] - // sel mask, target, target, falseReg - // - // This needs more careful thinking, so disabling it for now. - // test.ConditionalSelect_FalseOp(); + test.ConditionalSelect_FalseOp(); // Validates using inside ConditionalSelect with zero falseValue test.ConditionalSelect_ZeroOp(); diff --git a/src/tests/JIT/HardwareIntrinsics/Arm/Shared/SveGatherVectorFirstFaultingVectorBases.template b/src/tests/JIT/HardwareIntrinsics/Arm/Shared/SveGatherVectorFirstFaultingVectorBases.template index 5ba9b48264f1f..686b736a1563c 100644 --- a/src/tests/JIT/HardwareIntrinsics/Arm/Shared/SveGatherVectorFirstFaultingVectorBases.template +++ b/src/tests/JIT/HardwareIntrinsics/Arm/Shared/SveGatherVectorFirstFaultingVectorBases.template @@ -60,17 +60,7 @@ namespace JIT.HardwareIntrinsics.Arm test.RunStructFldScenario(); // Validates using inside ConditionalSelect with value falseValue - // Currently, using this operation in ConditionalSelect() gives incorrect result - // when falseReg == targetReg because this instruction uses Pg/Z to update the targetReg - // instead of Pg/M to merge it. As such, the value of falseReg is lost. Ideally, such - // instructions should be marked similar to RMW (a different flag name) to make sure that - // we do not assign falseReg/targetReg same. Then, we would do something like this: - // - // ldnf1sh target, pg/z, [x0] - // sel mask, target, target, falseReg - // - // This needs more careful thinking, so disabling it for now. - // test.ConditionalSelect_FalseOp(); + test.ConditionalSelect_FalseOp(); // Validates using inside ConditionalSelect with zero falseValue test.ConditionalSelect_ZeroOp();