Remove SetContained for GT_FIELD_LIST
authorHyeongseok Oh <hseok82.oh@samsung.com>
Mon, 28 Aug 2017 01:03:07 +0000 (10:03 +0900)
committerHyeongseok Oh <hseok82.oh@samsung.com>
Mon, 28 Aug 2017 01:03:07 +0000 (10:03 +0900)
Remove SetContained for GT_FIELD_LIST: already set when it is generated
Add comment

src/jit/lower.cpp

index 89c87cf..db51589 100644 (file)
@@ -860,7 +860,9 @@ GenTreePtr Lowering::NewPutArg(GenTreeCall* call, GenTreePtr arg, fgArgTabEntryP
 #endif // !FEATURE_UNIX_AMD64_STRUCT_PASSING
 
 #ifdef _TARGET_ARMARCH_
-    if (varTypeIsStruct(type) || arg->OperGet() == GT_FIELD_LIST)
+    // Mark contained when we pass struct
+    // GT_FIELD_LIST is always marked conatained when it is generated
+    if (varTypeIsStruct(type))
     {
         arg->SetContained();
         if ((arg->OperGet() == GT_OBJ) && (arg->AsObj()->Addr()->OperGet() == GT_LCL_VAR_ADDR))