From 23d664c18425a185541e8e6085c0dc91324508bb Mon Sep 17 00:00:00 2001 From: =?utf8?q?Michal=20Strehovsk=C3=BD?= Date: Thu, 14 Jul 2016 15:26:20 -0700 Subject: [PATCH] Remove assert validating intrinsic expansion on mustExpand The assert is incorrect. Consider expansion of CORINFO_INTRINSIC_RTH_GetValueInternal, where the RuntimeTypeHandle was created through a generic lookup helper call. The intrinsic expansion of this sequence becomes the generic lookup helper call on it's own. Commit migrated from https://github.com/dotnet/coreclr/commit/f063377de71efdba671832772507c1997bb3a046 --- src/coreclr/src/jit/importer.cpp | 7 ------- 1 file changed, 7 deletions(-) diff --git a/src/coreclr/src/jit/importer.cpp b/src/coreclr/src/jit/importer.cpp index 7c5a509..4e1ebee 100644 --- a/src/coreclr/src/jit/importer.cpp +++ b/src/coreclr/src/jit/importer.cpp @@ -3381,13 +3381,6 @@ InterlockedBinOpCommon: { NO_WAY("JIT must expand the intrinsic!"); } - else if ((retNode->gtFlags & GTF_CALL) != 0) - { - // If we must expand the intrinsic, - // retNode (the tree that corresponds to the intrinsic expansion) must be non-null, - // and the returned tree must not contain a call. - NO_WAY("JIT must not implement the intrinsic by a user call!"); - } } return retNode; -- 2.7.4