================
@@ -2300,6 +2300,9 @@ void CodeGenFunction::EmitStoreOfScalar(llvm::Value 
*Value, Address Addr,
         SmallVector<int, 16> Mask(NewVecTy->getNumElements(), -1);
         std::iota(Mask.begin(), Mask.begin() + VecTy->getNumElements(), 0);
         Value = Builder.CreateShuffleVector(Value, Mask, "extractVec");
+        // The extra lanes will be poison. Freeze the whole vector to make sure
+        // the padding memory is not poisoned, which may break coercion.
+        Value = Builder.CreateFreeze(Value);
----------------
fhahn wrote:

Happy to go with either. Not having the freeze is likely to be slightly better 
(e.g. we may be able to prove some lanes of the non-padding lanes could be 
poison), I just thought initially we might want to stay consistent w.r.t. 
shuffle with poison. 

Updated now

https://github.com/llvm/llvm-project/pull/164821
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to