JIT: don't create vector constants from relocatable constants (#107500)

We can't represent relocations in data currently. Fixes #107396. Co-authored-by: Andy Ayers <[email protected]> Co-authored-by: Jeff Schwartz <[email protected]>
dotnet · Sep 11, 2024 · fd36441 · fd36441
1 parent 972562c
commit fd36441
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/src/coreclr/jit/lower.cpp b/src/coreclr/jit/lower.cpp
@@ -8823,12 +8823,22 @@ void Lowering::LowerStoreIndirCoalescing(GenTreeIndir* ind)
         assert(prevData.IsStore());
         assert(currData.IsStore());
 
-        // For now, only constants are supported for data.
+        // For now, only non-relocatable constants are supported for data.
         if (!prevData.value->OperIsConst() || !currData.value->OperIsConst())
         {
             return;
         }
 
+        if (prevData.value->IsCnsIntOrI() && prevData.value->AsIntCon()->ImmedValNeedsReloc(comp))
+        {
+            return;
+        }
+
+        if (currData.value->IsCnsIntOrI() && currData.value->AsIntCon()->ImmedValNeedsReloc(comp))
+        {
+            return;
+        }
+
         // Otherwise, the difference between two offsets has to match the size of the type.
         // We don't support overlapping stores.
         if (abs(prevData.offset - currData.offset) != (int)genTypeSize(prevData.targetType))