diff --git a/src/coreclr/src/gc/gchandletable.cpp b/src/coreclr/src/gc/gchandletable.cpp index 9934889e58e4fe..3f69dbd1ef0356 100644 --- a/src/coreclr/src/gc/gchandletable.cpp +++ b/src/coreclr/src/gc/gchandletable.cpp @@ -99,7 +99,7 @@ IGCHandleStore* GCHandleManager::CreateHandleStore() return store; #else - assert("CreateHandleStore is not implemented when FEATURE_REDHAWK is defined!"); + assert(!"CreateHandleStore is not implemented when FEATURE_REDHAWK is defined!"); return nullptr; #endif } diff --git a/src/coreclr/src/jit/lowerxarch.cpp b/src/coreclr/src/jit/lowerxarch.cpp index 971424e64c96ac..28ac2ca2d236f1 100644 --- a/src/coreclr/src/jit/lowerxarch.cpp +++ b/src/coreclr/src/jit/lowerxarch.cpp @@ -5464,9 +5464,29 @@ void Lowering::ContainCheckHWIntrinsic(GenTreeHWIntrinsic* node) break; } + case NI_SSE2_ShiftLeftLogical128BitLane: + case NI_SSE2_ShiftRightLogical128BitLane: + case NI_AVX2_ShiftLeftLogical128BitLane: + case NI_AVX2_ShiftRightLogical128BitLane: + { +#if DEBUG + // These intrinsics should have been marked contained by the general-purpose handling earlier in the method. + + GenTree* lastOp = HWIntrinsicInfo::lookupLastOp(node); + assert(lastOp != nullptr); + + if (HWIntrinsicInfo::isImmOp(intrinsicId, lastOp) && lastOp->IsCnsIntOrI()) + { + assert(lastOp->isContained()); + } +#endif + + break; + } + default: { - assert("Unhandled containment for binary hardware intrinsic with immediate operand"); + assert(!"Unhandled containment for binary hardware intrinsic with immediate operand"); break; } } @@ -5643,7 +5663,7 @@ void Lowering::ContainCheckHWIntrinsic(GenTreeHWIntrinsic* node) default: { - assert("Unhandled containment for ternary hardware intrinsic with immediate operand"); + assert(!"Unhandled containment for ternary hardware intrinsic with immediate operand"); break; } }