heiher wrote:

Although the original intent of this patch was to resolve an assertion issue 
(https://github.com/ziglang/zig-bootstrap/issues/164#issuecomment-2332357069), 
it actually addresses two miscompilation issues 
(https://github.com/llvm/llvm-project/issues/97975 
https://github.com/llvm/llvm-project/issues/97981) as well. When the backport 
was initially proposed, I was believed that an ABI change would be necessary to 
ensure compatibility with future hardware supporting fp16. However, it's now 
clear that no such change will be required 
(https://github.com/llvm/llvm-project/pull/109368#issuecomment-2423879356). 
Given that this is primarily a bug fix, could we proceed with merging this 
backport into 19?

https://github.com/llvm/llvm-project/pull/109093
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to