[RISC-V][JIT] Disable promotion for splitted args (#87855)

clamp03 · web-flow · commit 0633ecfb79a3 · 2023-06-25T15:37:48.000+02:00
* [RISC-V][JIT] Reset lvIsRegArg in splitted arg

* Remove LA

* Updated based on the review

* Moved codes
diff --git a/src/coreclr/jit/lclvars.cpp b/src/coreclr/jit/lclvars.cpp
@@ -2175,6 +2175,13 @@ bool Compiler::StructPromotionHelper::ShouldPromoteStructVar(unsigned lclNum)
                         lclNum);
                 shouldPromote = false;
             }
+#if defined(TARGET_LOONGARCH64) || defined(TARGET_RISCV64)
+            else if (varDsc->lvIsSplit)
+            {
+                JITDUMP("Not promoting multireg struct local V%02u, because it is splitted.\n", lclNum);
+                shouldPromote = false;
+            }
+#endif // TARGET_LOONGARCH64 || TARGET_RISCV64
         }
         else
 #endif // !FEATURE_MULTIREG_STRUCT_PROMOTE

Original file line number	Diff line number	Diff line change
`@@ -2175,6 +2175,13 @@ bool Compiler::StructPromotionHelper::ShouldPromoteStructVar(unsigned lclNum)`
`2175`	`2175`	`lclNum);`
`2176`	`2176`	`shouldPromote = false;`
`2177`	`2177`	`}`
	`2178`	`+#if defined(TARGET_LOONGARCH64) \|\| defined(TARGET_RISCV64)`
	`2179`	`+ else if (varDsc->lvIsSplit)`
	`2180`	`+ {`
	`2181`	`+ JITDUMP("Not promoting multireg struct local V%02u, because it is splitted.\n", lclNum);`
	`2182`	`+ shouldPromote = false;`
	`2183`	`+ }`
	`2184`	`+#endif // TARGET_LOONGARCH64 \|\| TARGET_RISCV64`
`2178`	`2185`	`}`
`2179`	`2186`	`else`
`2180`	`2187`	`#endif // !FEATURE_MULTIREG_STRUCT_PROMOTE`